RealDevSquad / RealDevSquad/website-backend
Remove Backfill Script for "in_discord" False Role
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 74
- Forks
- 276
- Avg merge
- 1d 26m
- Merged PRs (30d)
- 14
Description
Issue: Remove Backfill Script for "in_discord" False Role
Description
Currently, there is a backfill script in place that adds the "in_discord" role with a value of false to all existing users. This approach was used to initialize the "in_discord" role before implementing verification. Now that we have a robust verification process, this backfill script is no longer necessary and can be safely removed.
Problem
The backfill script for the "in_discord" false role adds unnecessary complexity to our user management system. It may lead to confusion and unintended behavior, especially with the new verification process in place.
Proposed Solution
We should remove the backfill script that sets the "in_discord" role to false for all users. Instead, the "in_discord" role will be assigned based on the outcome of the verification process.
Benefits
- Simpler Codebase: Removing the backfill script simplifies our codebase by eliminating redundant and unnecessary processes.
- Enhanced User Clarity: Users' "in_discord" status will accurately reflect their verification status without any manual intervention.
- Improved Maintenance: The removal of unnecessary scripts reduces maintenance efforts and potential sources of errors.
Implementation Steps
- Identify the location and logic of the backfill script.
- Safely remove the script from the codebase.
Acceptance Criteria
- The backfill script that adds the "in_discord" false role to existing users is successfully removed.
- Users' "in_discord" role is assigned accurately based on the verification process.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Search the repository for the "in_discord" role and the backfill logic, then read the verification flow that now assigns this role. Remove only the obsolete false-value backfill and run the relevant user-management or verification tests to confirm the role is still assigned from verification.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, nodejs
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100