Northeastern-Electric-Racing / Northeastern-Electric-Racing/FinishLine
[Slack Notification] - Toggle Slack notification for CR creation
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 36
- Forks
- 9
- Avg merge
- 4d 21h
- Merged PRs (30d)
- 11
Description
Desired Changes
Currently when you create a change request, it sends a slack notification to everyone. We want to make it so that you can toggle weather or not to send the slack notification when creating a CR.
Potential solution
- Add a toggle on the new CR page / modal and pass in a new boolean to the create CR endpoint
- You will have to test and might need slack env variables. Ask Sean or Aaryan about how to test it if you get stuck
Acceptance Criteria
- The create CR endpoint has been updated to pass in a new boolean that determines whether or not to send the slack notification
- There is a toggle on the create CR page / modal
Screenshots (as needed)
No response
Contributor guide
No contributing guide indexed for this repository
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
Start at the create CR page or modal and trace the create CR endpoint, then identify where the existing Slack notification is triggered. Add the requested toggle and boolean behavior so creation can conditionally send the notification, and verify both paths against the acceptance criteria. Ask Sean or Aaryan about Slack environment variables if testing is blocked.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- full-stack
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100