Skip to content
This repository was archived by the owner on Mar 5, 2026. It is now read-only.

Fix extensionConfiguration/change type, respect authStatus from cody …#2629

Merged
kalanchan merged 1 commit intojb-v7.1.xfrom
kalan/cp-2623
Nov 12, 2024
Merged

Fix extensionConfiguration/change type, respect authStatus from cody …#2629
kalanchan merged 1 commit intojb-v7.1.xfrom
kalan/cp-2623

Conversation

@kalanchan
Copy link
Copy Markdown
Contributor

cherry picking (#2623)

Changes

  1. Bumped cody commit to match the one from jb-v7.1.x branch
  2. Fixed "extensionConfiguration/change" endpoint type - issue was introduced
    there
  3. For the sake of correctness I also added check of returned auth state and if auth fails I set the token as invalid.

Test plan

  1. Run IDE
  2. Remove all account in the cody settings
  3. Login panel shoul appear
  4. Sign in to enterprise account
  5. Make sure chat panel appears and in the account panel enterprise acc is shown
  6. Remove all account in the cody settings
  7. Login panel should appear
  8. Sign in to free dotcom account
  9. Make sure chat panel appears and in the account panel free acc is shown

Test plan

…2623)

## Changes

1. Bumped cody commit to match the one from `jb-v7.1.x` branch
2. Fixed `"extensionConfiguration/change"` endpoint type - issue was
introduced
[there](8ea63fc)
3. For the sake of correctness I also added check of returned auth state
and if auth fails I set the token as invalid.

## Test plan

1. Run IDE
4. Remove all account in the cody settings
5. Login panel shoul appear
6. Sign in to enterprise account 
7. Make sure chat panel appears and in the account panel enterprise acc
is shown
8. Remove all account in the cody settings
9. Login panel should appear
10. Sign in to free dotcom account
11. Make sure chat panel appears and in the account panel free acc is
shown
@kalanchan kalanchan merged commit 7bae16d into jb-v7.1.x Nov 12, 2024
@kalanchan kalanchan deleted the kalan/cp-2623 branch November 12, 2024 20:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants