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

Setup a basic view for task api and add query params validator #11

Open
wants to merge 7 commits into
base: feat-get-todo-api-add-repository
Choose a base branch
from

Conversation

samarpan1738
Copy link
Contributor

@samarpan1738 samarpan1738 commented Dec 20, 2024

Date: 20 Dec 2024

Developer Name: Samarpan Harit



Issue Ticket Number

Description

  1. Restructure folder for views and it's tests
  2. Add version in base api url
  3. Add a basic view for get task API
  4. Add query param validator for get task API
  5. Add tests for query param validator

Documentation Updated?

  • Yes
  • No

Under Feature Flag

  • Yes
  • No

Database Changes

  • Yes
  • No

Breaking Changes

  • Yes
  • No

Development Tested?

  • Yes
  • No

Screenshots

Screenshot 1

Test Coverage

Coverage report Screenshot 2024-12-21 at 12 10 13 AM

Additional Notes

The CI is failing right now because there an issue in the workflow. I have raised a PR to fix this. Once it's merged the CI should pass

@samarpan1738 samarpan1738 force-pushed the feat-get-todo-api-view-and-validator branch from 074fa71 to f91df5a Compare December 22, 2024 19:38
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.

1 participant