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

fix(UI-1321): fix slack oauth edit #984

Merged
merged 1 commit into from
Feb 11, 2025
Merged

fix(UI-1321): fix slack oauth edit #984

merged 1 commit into from
Feb 11, 2025

Conversation

RonenMars
Copy link
Contributor

Description

Can't edit Slack connections

Observed in my local server (head = main branch), not in prod.

After initializing a Slack OAuth connection, the auth_type is set (by the frontend, not the backend) to "oauthDefault". This is fine, but the web UI (v2.135.0) doesn't know this value yet, so if the user tries to edit the connection, they see this:

image

Linear Ticket

https://linear.app/autokitteh/issue/UI-1321/cant-edit-slack-connections

What type of PR is this? (check all applicable)

  • 💡 (feat) - A new feature (non-breaking change which adds functionality)
  • 🔄 (refactor) - Code Refactoring - A code change that neither fixes a bug nor adds a feature
  • 🐞 (fix) - Bug Fix (non-breaking change which fixes an issue)
  • 🏎 (perf) - Optimization
  • 📄 (docs) - Documentation - Documentation only changes
  • 📄 (test) - Tests - Adding missing tests or correcting existing tests
  • 🎨 (style) - Styles - Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
  • ⚙️ (ci) - Continuous Integrations - Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
  • ☑️ (chore) - Chores - Other changes that don't modify src or test files
  • ↩️ (revert) - Reverts - Reverts a previous commit(s).

@RonenMars RonenMars requested a review from J1za February 11, 2025 06:37
@RonenMars RonenMars enabled auto-merge (squash) February 11, 2025 09:22
@RonenMars RonenMars force-pushed the ronen/fix/slack-oauth branch from 763f37f to a241881 Compare February 11, 2025 09:22
@RonenMars RonenMars merged commit ceae242 into main Feb 11, 2025
38 checks passed
@RonenMars RonenMars deleted the ronen/fix/slack-oauth branch February 11, 2025 09:28
Copy link

🎉 This PR is included in version 2.140.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants