Skip to content
This repository has been archived by the owner on Jun 14, 2024. It is now read-only.

[TASK] Migrate GET /api/_info endpoint to GET /api/v1/version #161

Closed
Tracked by #4773
jfcalvo opened this issue May 9, 2024 · 0 comments · Fixed by #162
Closed
Tracked by #4773

[TASK] Migrate GET /api/_info endpoint to GET /api/v1/version #161

jfcalvo opened this issue May 9, 2024 · 0 comments · Fixed by #162
Assignees

Comments

@jfcalvo
Copy link
Member

jfcalvo commented May 9, 2024

No description provided.

@jfcalvo jfcalvo self-assigned this May 9, 2024
@jfcalvo jfcalvo transferred this issue from argilla-io/argilla May 9, 2024
@jfcalvo jfcalvo linked a pull request May 9, 2024 that will close this issue
15 tasks
jfcalvo added a commit that referenced this issue May 13, 2024
# Description

With this PR I'm creating a new endpoint on API v1 `GET /api/v1/version`
to return the current argilla version.

I renamed it to `/api/v1/version` instead of `/api/v1/info` because at
the end we are only returning the version.

Closes #161

**Type of change**

(Please delete options that are not relevant. Remember to title the PR
according to the type of change)

- [ ] Bug fix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- [ ] Refactor (change restructuring the codebase without changing
functionality)
- [ ] Improvement (change adding some improvement to an existing
functionality)
- [ ] Documentation update

**How Has This Been Tested**

- [x] Added new tests.

**Checklist**

- [ ] I added relevant documentation
- [ ] follows the style guidelines of this project
- [ ] I did a self-review of my code
- [ ] I made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] I filled out [the contributor form](https://tally.so/r/n9XrxK)
(see text above)
- [ ] I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)
@jfcalvo jfcalvo closed this as completed May 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant