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

#33 Issue PR #36

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from
Open

#33 Issue PR #36

wants to merge 4 commits into from

Conversation

gauravrmt
Copy link
Collaborator

Fixes:

  • removed fetching data from file.
  • fetched all data from database.
  • fixed and added new test cases for /geography/country api.
  • fixed swagger api documentation and added new response documentation for the same.

sahityagaurav4210 and others added 4 commits March 5, 2025 12:08
### Fixes
- partial changes
### Changes
- added `country` and `state` model.
- added seeders for added model.
### Fixes:
- removed fetching data from file.
- fetched all data from database.
- fixed and added new `test cases`  for /geography/country api.
- fixed `swagger api` documentation and added new response documentation for the same.
@gauravrmt gauravrmt added bug Something isn't working refactorings Code and other refactorings labels Mar 12, 2025
@gauravrmt gauravrmt self-assigned this Mar 12, 2025
@gauravrmt gauravrmt linked an issue Mar 12, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working refactorings Code and other refactorings
Projects
Status: Backlog
Development

Successfully merging this pull request may close these issues.

Fix currency and country apis
2 participants