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

Refactor: Slack Message #16

Open
1 of 3 tasks
abidishajia opened this issue Jun 13, 2024 · 1 comment
Open
1 of 3 tasks

Refactor: Slack Message #16

abidishajia opened this issue Jun 13, 2024 · 1 comment
Labels
Priority: Low This is of low importance. Take care of other issues before this.

Comments

@abidishajia
Copy link
Collaborator

abidishajia commented Jun 13, 2024

We currently have a Slack folder with a sendMessage function. It is used for the elex-testing-calendar bot.

  • Move the Slack folder to the root directory.
  • Refactor the code to pass the message as an argument and use the function for both the bots.
  • For the elex-tabulation-data bot, only send a message to the channel when there is an update. The format could be something like:
  • The race call status for X, Y, and Z races have changed to Called.
  • The tabulation status for X, Y, and Z has been changed to Active Tabulation.
@abidishajia abidishajia changed the title Slack Message Refactor: Slack Message Jun 14, 2024
@abidishajia
Copy link
Collaborator Author

Since the notifications for tabulationStatus and raceCallStatus are very complex, it is best to leave them separate. Maybe the other way would be to format a message in different files, and then just pass in the message as an argument instead of the actual data.

@abidishajia abidishajia added the Priority: Low This is of low importance. Take care of other issues before this. label Jun 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Low This is of low importance. Take care of other issues before this.
Projects
None yet
Development

No branches or pull requests

1 participant