Skip to content

Remove deprecated /indicators/base API aliases#624

Open
Copilot wants to merge 2 commits intomainfrom
copilot/fix-604
Open

Remove deprecated /indicators/base API aliases#624
Copilot wants to merge 2 commits intomainfrom
copilot/fix-604

Conversation

Copy link
Contributor

Copilot AI commented Aug 26, 2025

This PR removes the deprecated /indicators/base API route aliases as requested in the issue, now that NCR and Prefect workflows have been updated to use the /indicators/cmip5 endpoints.

Changes Made

  • Removed route decorators for /indicators/base/point/<lat>/<lon> and /indicators/base/area/<var_id> from their respective functions in routes/indicators.py
  • Updated cached URLs list in luts.py to remove the deprecated /indicators/base/point/ and /indicators/base/area/ entries
  • Cleaned up module docstring to remove outdated reference to "base" indicators

The API now exclusively serves climate indicators data through the /indicators/cmip5/* endpoints, which match the current API documentation. The underlying functionality remains unchanged - only the deprecated URL aliases have been removed.

Impact

  • /indicators/base/point/<lat>/<lon>removed (use /indicators/cmip5/point/<lat>/<lon>/ instead)
  • /indicators/base/area/<var_id>removed (use /indicators/cmip5/area/<var_id>/ instead)
  • /indicators/cmip5/* endpoints → unchanged and functional

Fixes #604.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copy link
Contributor Author

Copilot AI commented Aug 26, 2025

@brucecrevensten 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs.

I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review.

Co-authored-by: brucecrevensten <525049+brucecrevensten@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove alias for /indicators/base from API Remove deprecated /indicators/base API aliases Aug 26, 2025
Copilot AI requested a review from brucecrevensten August 26, 2025 21:58
@BobTorgerson BobTorgerson marked this pull request as ready for review November 20, 2025 19:27
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.

Remove alias for /indicators/base from API

2 participants