-
Notifications
You must be signed in to change notification settings - Fork 291
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
feat: Export users in backups #15072
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #15072 +/- ##
=======================================
Coverage 43.20% 43.21%
=======================================
Files 645 645
Lines 21613 21632 +19
Branches 4956 4959 +3
=======================================
+ Hits 9338 9348 +10
- Misses 11090 11099 +9
Partials 1185 1185 |
d8e4ef4
to
c4b4152
Compare
5f1a018
to
761a9e8
Compare
@@ -71,6 +72,57 @@ const messages = [ | |||
}, | |||
]; | |||
|
|||
const users = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't have user generator? 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Damn, you are completely right. Good call 👍
usersChunk, | ||
user => constructUserPrimaryKey(user.qualified_id), | ||
); | ||
importedEntities += usersChunk.length; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Move usersChunk.length to variable. You using it there multiple times :)
This is the first part of saving users' metadata in backups.
This will export the data (but not load it yet, this will be done in a followup PR)