fix: current device logout showing error#40351
fix: current device logout showing error#40351juliajforesti wants to merge 2 commits intodevelopfrom
Conversation
Co-authored-by: Copilot <copilot@github.com>
|
Looks like this PR is not ready to merge, because of the following issues:
Please fix the issues and try again If you have any trouble, please check the PR guidelines |
🦋 Changeset detectedLatest commit: ec33ad7 The changes in this PR will be included in the next version bump. This PR includes changesets to release 41 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## develop #40351 +/- ##
===========================================
- Coverage 69.86% 69.85% -0.02%
===========================================
Files 3298 3298
Lines 119347 119357 +10
Branches 21530 21522 -8
===========================================
- Hits 83377 83371 -6
- Misses 32672 32684 +12
- Partials 3298 3302 +4
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
CORE-2033
Proposed changes (including videos or screenshots)
issue:
/v1/sessions/logout.metriggered a refetch of the device listsolution:
/v1/sessions/logout.me) vs logging out other users' sessionsthrowOnError: falseto the mutation to prevent errors from reaching React error boundariesIssue(s)
Steps to test or reproduce
Further comments