Repository navigation
Conversation
Done so we can access anywhere how we structure our workspace credentials
|
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: 865a9d4 The changes in this PR will be included in the next version bump. This PR includes changesets to release 32 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 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #32122 +/- ##
===========================================
+ Coverage 55.59% 55.65% +0.05%
===========================================
Files 2407 2429 +22
Lines 52905 53396 +491
Branches 10866 10983 +117
===========================================
+ Hits 29415 29717 +302
- Misses 20886 21045 +159
- Partials 2604 2634 +30
Flags with carried forward coverage won't be shown. Click here to find out more. |
There was a problem hiding this comment.
I believe we're trying to only add migrations in the next major release 😬
Is there any chance we could move this to the release-7.0.0 branch?
cc @sampaiodiego
Co-authored-by: Matheus Barbosa Silva <36537004+matheusbsilva137@users.noreply.github.com>
This PR is a children of: #31986 so it must be merged before this one gets merged.
Proposed changes (including videos or screenshots)
Since all the cloud credentials data must be stored inside the
rocketchat_workspace_credentialscollection, we must migrate all the values from inside thesettingscollection to their new home.Issue(s)
Steps to test or reproduce
Further comments
CONN-5