Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add warning that Impact/Segment OAuth must be done by workspace owner #6474

Merged
merged 2 commits into from
Nov 26, 2024

Conversation

ashton-huxtable
Copy link
Contributor

Proposed changes

Added a warning in the setup that OAuth for this destination must be completed by workspace owners. A customer ran into issues with this and the UI and console did not surface any significant warnings to explain why.

Merge timing

  • ASAP once approved

Related issues (optional)

n/a

@ashton-huxtable ashton-huxtable marked this pull request as ready for review August 21, 2024 15:52
@ashton-huxtable ashton-huxtable requested a review from a team as a code owner August 21, 2024 15:52
@ashton-huxtable ashton-huxtable requested a review from pwseg August 21, 2024 15:52
@pwseg pwseg changed the title Add warning that OAuth must be done by workspace owner Add warning that Impact/Segment OAuth must be done by workspace owner Nov 26, 2024
@pwseg pwseg added the content-update updates to content that are not new features, includes grammar fixes, added notes label Nov 26, 2024
@pwseg pwseg merged commit ea6a01f into develop Nov 26, 2024
4 checks passed
@pwseg pwseg deleted the ashton-huxtable-patch-2 branch November 26, 2024 23:40
Copy link
Contributor

Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content-update updates to content that are not new features, includes grammar fixes, added notes KCS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants