feat: send xgeeks board phase to slack #1156
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Button to start voting phase sends webSocket to the backend. Backend sends webSocket to frontend.
when phase is updated sends websocket to boardId
change updateBoardPhase to accept phase of the board. Remove socketId from AlertVotePhase change fetch to boardService
Button show when all sub-boards merged and phase in "addcards"
Frontend changes -updateBoardPhaseRequest changed the way it sends the body -On board.ts changed the id name be the same as the endpoint dto Backend changes -add BoardPhaseDto to the endpoint
When board phase updated, the team is xgeeks and slack is enable sends message to the xgeeks SLACK_MASTER_CHANNEL_ID
nunocaseiro
requested changes
Feb 27, 2023
backend/src/modules/communication/applications/slack-communication.application.ts
Outdated
Show resolved
Hide resolved
backend/src/modules/communication/consumers/slack-send-message.consumer.ts
Outdated
Show resolved
Hide resolved
backend/src/modules/communication/consumers/slack-send-message.consumer.ts
Outdated
Show resolved
Hide resolved
backend/src/modules/communication/interfaces/communication.application.interface.ts
Outdated
Show resolved
Hide resolved
nunocaseiro
approved these changes
Feb 28, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Relates to #1048
Screenshots (if visual changes)
Proposed Changes
This pull request closes #1048