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

Implemented Queue #36

Merged
merged 14 commits into from
Jan 26, 2025
Merged

Implemented Queue #36

merged 14 commits into from
Jan 26, 2025

Conversation

joyguptaa
Copy link
Contributor

Date: 23 Jan 2024

Developer Name: Joy Gupta


Issue Ticket Number

Description

Added code for queue, added method to send message to queue

Documentation Updated?

  • Yes
  • No

Under Feature Flag

  • Yes
  • No

Database Changes

  • Yes
  • No

Breaking Changes

  • Yes
  • No

Development Tested?

  • Yes
  • No

Test Coverage

Screenshot 2025-01-23 at 10 53 23 PM Screenshot 2025-01-23 at 10 53 34 PM

Copy link
Member

@iamitprakash iamitprakash left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please updated documentation, to onboard new dev on this task

@joyguptaa
Copy link
Contributor Author

QUEUE_NAME

Hello @iamitprakash
The documentation on how to set up the queue will be incorporated into the service broker itself. Once I update its README file, I'll also add a link to this repository.

@iamitprakash
Copy link
Member

QUEUE_NAME

Hello @iamitprakash The documentation on how to set up the queue will be incorporated into the service broker itself. Once I update its README file, I'll also add a link to this repository.

add details to spin up in local at least like you are using in your machine @joyguptaa

@joyguptaa
Copy link
Contributor Author

QUEUE_NAME

Hello @iamitprakash The documentation on how to set up the queue will be incorporated into the service broker itself. Once I update its README file, I'll also add a link to this repository.

add details to spin up in local at least like you are using in your machine @joyguptaa

done

@joyguptaa joyguptaa requested a review from yesyash January 25, 2025 06:29
iamitprakash
iamitprakash previously approved these changes Jan 25, 2025
@iamitprakash iamitprakash merged commit ae92f9f into develop Jan 26, 2025
2 checks passed
@iamitprakash iamitprakash deleted the queue branch January 26, 2025 18:56
iamitprakash added a commit that referenced this pull request Feb 1, 2025
* Implemented Queue (#36)

* Chore : Updated .env.example

* Feat : Added rabbitmq

* Feat : Added exponential retry method

* Feat: Added DTO for message

* Bug : Fixed ExponentialBackoffRetry function

* Feat : Added ExponentialBackoffRetry helper

* Feat : Added Queue code

* Test : Updated Test

* Feat : Updated config to load queue name

* Feat : Added QUEUE_URL in env

* Doc : Updated readme

* Chore : Updated order

* Chore : Fixed broken test and added error handling

* Feat : Added queue handler (#37)

* Chore : Updated .env.example

* Feat : Added rabbitmq

* Feat : Added exponential retry method

* Feat: Added DTO for message

* Bug : Fixed ExponentialBackoffRetry function

* Feat : Added ExponentialBackoffRetry helper

* Feat : Added Queue code

* Test : Updated Test

* Feat : Updated config to load queue name

* Feat : Added QUEUE_URL in env

* Doc : Updated readme

* Chore : Updated order

* Chore : Fixed broken test and added error handling

* Feat : Added queue handler

* Add QUEUE_URL and QUEUE_NAME env  (#39)

* Chore : Updated .env.example

* Feat : Added rabbitmq

* Feat : Added exponential retry method

* Feat: Added DTO for message

* Bug : Fixed ExponentialBackoffRetry function

* Feat : Added ExponentialBackoffRetry helper

* Feat : Added Queue code

* Test : Updated Test

* Feat : Updated config to load queue name

* Feat : Added QUEUE_URL in env

* Doc : Updated readme

* Chore : Updated order

* Chore : Fixed broken test and added error handling

* Feat : Added queue handler

* chore: add queuename and url env

---------

Co-authored-by: Joy <[email protected]>

* replace env to vars (#41)

* Feat/listening command (#43)

* Feat : Added listening command

* Feat : Added helper to DataPacket

* Feat : Updated SendMessage Code

* Test : Fixed test for DataPacket

* Feat : Added listening service

* Chore : Updated function declaration

* Chore : Updated InteractionResponseData

* Feat : Addded end-to-end implementation of listening command

* Feat : Build common package for discord

* WIP : Creating methods for discord

* Test : Added test for commandHandler.MainHandler in QueueHandler

* Test : Added test for QueueHandler

* Test : Added test for MainHandler

* Test : Added test

* Test : Added test

* Chore : Removed consoles

* Chore : Removed fmt

* Chore : Minor change

* Chore : Minor change

* feat : Moved register command flow within main.go (#45)

* Feat : Moved register command flow within main.go

* Refactor : Updated vars & functions name

* Update commands/register/register.go

Co-authored-by: Yash Raj <[email protected]>

* Chore : Minor change

* Chore : Minor change

---------

Co-authored-by: Amit Prakash <[email protected]>
Co-authored-by: Yash Raj <[email protected]>

---------

Co-authored-by: Prakash Choudhary <[email protected]>
Co-authored-by: Amit Prakash <[email protected]>
Co-authored-by: Yash Raj <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants