Skip to content

Conversation

@tanhongit
Copy link
Member

@tanhongit tanhongit commented Feb 15, 2025

Summary

  • Refactor Webhook service
  • Handle send comments with queue job

Unit Testing

Operation check

Caution

Release Notes

Note

Additional Notes

Checklist

  • Code follows our Coding Standards.
  • Code matches the confirmed solution in the ticket.
  • Unit/Integration tests have been added or updated.
  • Env/Config/DB changes were confirmed by @cslant/admin.

@tanhongit tanhongit added the feature request Want to create a new feature, improvement, or change in the existing system or project label Feb 15, 2025
@tanhongit tanhongit self-assigned this Feb 15, 2025
@tanhongit tanhongit marked this pull request as draft February 15, 2025 03:34
tanhongit and others added 22 commits February 15, 2025 10:38
# Conflicts:
#	src/Services/WebhookService.php
…asses by removing unnecessary logging and GitHub service dependency injection
@tanhongit tanhongit requested a review from mytruong-z February 19, 2025 14:24
@tanhongit tanhongit marked this pull request as ready for review February 19, 2025 14:25
@cslant-bot
Copy link

Status has been set to $${\color{GRAY}Backlog}$$.

Priority has been set to $${\color{YELLOW}Medium}$$.

Size has been set to $${\color{YELLOW}M}$$.

Size has been changed from $${\color{YELLOW}M}$$ to $${\color{ORANGE}L}$$.

Made changes by: github-project-automation[bot]

@cslant-bot
Copy link

Estimate has been set to 8.

Made changes by: tanhongit

@cslant-bot
Copy link

Start date has been set to 2025-02-15.

End date has been set to 2025-02-19.

Made changes by: tanhongit

@mytruong-z mytruong-z merged commit 95a9c7e into main Feb 19, 2025
14 checks passed
@mytruong-z mytruong-z deleted the queue-event branch February 19, 2025 14:44
@cslant-bot
Copy link

Status has been changed from $${\color{GRAY}Backlog}$$ to $${\color{PURPLE}Done}$$.

Made changes by: github-project-automation[bot]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature request Want to create a new feature, improvement, or change in the existing system or project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants