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

Add issue count methods #7

Merged
merged 5 commits into from
Jun 7, 2024
Merged

Conversation

patrickpa
Copy link
Contributor

Description

This PR adds the issue count methods and also the id manager to prevent users of creating issues with same id.

Main changes

  1. Add issue count methods
  2. Add id manager for managing issues ids

How was the PR tested?

  1. Unit-test are passing with the new additions.

Notes

patrickpa added 3 commits June 7, 2024 11:51
Signed-off-by: patrickpa <patrick@ivex.ai>
Signed-off-by: patrickpa <patrick@ivex.ai>
Signed-off-by: patrickpa <patrick@ivex.ai>
@patrickpa patrickpa requested a review from hoangtungdinh June 7, 2024 09:54
@patrickpa patrickpa self-assigned this Jun 7, 2024
patrickpa added 2 commits June 7, 2024 12:09
Signed-off-by: patrickpa <patrick@ivex.ai>
Signed-off-by: patrickpa <patrick@ivex.ai>
Copy link
Contributor

@hoangtungdinh hoangtungdinh left a comment

Choose a reason for hiding this comment

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

Thanks @patrickpa

@hoangtungdinh hoangtungdinh merged commit e671003 into develop Jun 7, 2024
7 checks passed
@hoangtungdinh hoangtungdinh deleted the patrick/add_issue_count_methods branch June 7, 2024 10:13
hoangtungdinh pushed a commit that referenced this pull request Jun 18, 2024
Signed-off-by: patrickpa <patrick@ivex.ai>
hoangtungdinh pushed a commit that referenced this pull request Jun 18, 2024
Signed-off-by: patrickpa <patrick@ivex.ai>
Signed-off-by: Patrick Abrahão <patrickabrahao.pa@gmail.com>
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.

2 participants