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

feat: Added poolID as a req parameter for the mentioned routes in the issue #302 #303

Closed
wants to merge 8 commits into from

Conversation

webbdays
Copy link

Before submitting this PR, please make sure:

  • [Yes] Your code builds clean without any errors or warnings
  • [Yes] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
Related to #302
Added poolID as a req parameter for the mentioned routes.

Tests performed by the developer:
yarn test:unit
But some tests are failing because of adding new req param.
Yet to write unit tests for the new param.

@webbdays webbdays marked this pull request as ready for review March 31, 2024 12:23
@nikspz
Copy link
Contributor

nikspz commented Apr 1, 2024

hi @webbdays, PR should be cloned from development and changes should be pointed to development branch

You could close this PR and create new or made changes here

@nikspz nikspz changed the base branch from main to development April 1, 2024 14:22
@webbdays
Copy link
Author

webbdays commented Apr 1, 2024

Sure.

@nikspz nikspz closed this Apr 2, 2024
@nikspz
Copy link
Contributor

nikspz commented Apr 2, 2024

Closed since it was created from main branch (not development)

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.

None yet

3 participants