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

Script to migrate user roles #351

Merged
merged 3 commits into from
Jun 8, 2021
Merged

Conversation

sumitd94
Copy link
Contributor

This PR contains the migration script, which will be running only once which will remove the isMember field, and add a new field roles which will be an object something like this

{
 roles: { member: true }
}

All those users who have isMember set to true, will have member: true in the roles objects, and for the rest of all users, member: false

endpoint - https://api.realdevsquad.com/members/member-to-role-migration

@ankurnarkhede
Copy link
Contributor

@sumitd94 , executing this will solve the failing tests in #296, right?
Also, is there any other way to execute this query or adding the route is the only way?

Copy link
Contributor

@ankurnarkhede ankurnarkhede left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ✅

@ankurnarkhede ankurnarkhede merged commit 4b2a081 into develop Jun 8, 2021
@ankurnarkhede ankurnarkhede deleted the migrate-member-userRole branch June 8, 2021 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants