Uh oh!
There was an error while loading. Please reload this page.
Feature/integ 3159/trust updates - #170
Conversation
| ::: incydr.enums.trusted_activities.BrowserDestination | ||
| :docstring: | ||
| * **AIRTABLE** = `"AIRTABLE"` |
There was a problem hiding this comment.
are we going to need to update these every time there is a new trusted activity browser destination? Are we going to be notified by the trusted activities team saying "hey, new browser destination just dropped"?
There was a problem hiding this comment.
alternatively, do we need to enum these out? Could we just let it rip as a string and they need to find the valid values out from... elsewhere? The trusted activity api docs, perhaps?
There was a problem hiding this comment.
We could just let it be a string. The permissible values are documented in the API documentation. I went with the enum because that's the pattern we're already doing for cloud services and email services, so I wanted to be consistent with it.
| ::: incydr.enums.trusted_activities.BrowserDestination | ||
| :docstring: | ||
| * **AIRTABLE** = `"AIRTABLE"` |
There was a problem hiding this comment.
alternatively, do we need to enum these out? Could we just let it rip as a string and they need to find the valid values out from... elsewhere? The trusted activity api docs, perhaps?
8474aef
into
feature/INTEG-3142/session-typeUh oh!
There was an error while loading. Please reload this page.
Adds some new features to trust