Northeastern-Electric-Racing / Northeastern-Electric-Racing/FinishLine

Slack Testing - Make env variable to disable/ enable slack testing on dev

Open
#2,368 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

back-end medium technical improvement
Dominant language
TypeScript
Stars
36
Forks
9
Avg merge
4d 21h
Merged PRs (30d)
11

Description

Description

Right now we have a check everywhere a slack message is sent for whether we're on prod. If we're on prod, we send the message, if we aren't, we don't. This ticket is to add in another environment variable that allows us to toggle slackbot testing on and off.

Acceptance Criteria

There's a new environment variable `SLACKBOT_TESTING' that when set to 'ENABLED' allows slack messages to be sent on your local.

Proposed Solution

Add a check everywhere there's a check for prod for slack messages that bypasses it if this environment variable is set to 'ENABLED'

Mocks

N/A

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Search the codebase for the existing production checks around Slack message sends and inspect how environment variables are configured. Implement the requested behavior across those checks, then verify that SLACKBOT_TESTING set to ENABLED permits local Slack messages while the default behavior remains unchanged.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
backend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.