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

Update Get Recordset Change Route #1367

Open
Aravindh-Raju opened this issue May 23, 2024 · 0 comments · May be fixed by #1368
Open

Update Get Recordset Change Route #1367

Aravindh-Raju opened this issue May 23, 2024 · 0 comments · May be fixed by #1368
Assignees
Labels
area/api Items relating to the API kind/maintenance Code cleanup, refactoring, upgrading to new shiny things

Comments

@Aravindh-Raju
Copy link
Member

Aravindh-Raju commented May 23, 2024

Currently the Get Recordset Change has a Segment which is of no use. It's not passed on to any service methods or used anywhere. We should remove that to avoid any confusion.

For eg in the below api, the segment next to recordsets is of no use. It can be anything like dummy, test or whatever and it'll still work as it is not used anywhere or provide any use:

https://localhost:9000/zones/3db0d8y4-967c-46fa-9560-560b11m5cb38/recordsets/{this can be anything. it doesn't matter}/changes/09950016-14bd-469a-bc62-9e8b0zlka69
@Aravindh-Raju Aravindh-Raju added kind/maintenance Code cleanup, refactoring, upgrading to new shiny things area/api Items relating to the API labels May 23, 2024
@Aravindh-Raju Aravindh-Raju self-assigned this May 23, 2024
@Aravindh-Raju Aravindh-Raju linked a pull request May 23, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/api Items relating to the API kind/maintenance Code cleanup, refactoring, upgrading to new shiny things
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant