NavigaTUM icon indicating copy to clipboard operation
NavigaTUM copied to clipboard

Announcement issue for breaking API changes

Open CommanderStorm opened this issue 3 years ago • 7 comments

This Issue tracks:

  • who uses the API,
  • what features you are using and
  • what change-timelines ("I need 3 Weeks at minimum, because ...") you would prefer.

If you have implemented a part of the API, please note down in the comments which ones. Doing so will indicate, that we will have to keep this API in a stable state and cannot do breaking changes without the change-timeline you require.

We will use this Issue to note breaking API changes.

There are some things about compatibility and assumptions we are making:

  • Adding fields should never break your Application
  • If we differ from the openapi spec this is a bug! Please report it ^^
  • Undocumented behaviour is not relied upon (tell us if something is undocumented)

CommanderStorm avatar May 25 '22 23:05 CommanderStorm

We have renamed nb_hits to estimatedTotalHits in https://github.com/TUM-Dev/navigatum/pull/162/commits/9564038ae53dc7e2f6b849dc3c33e3c0eb9e7bb1

Since nobody has commented on this issue, this is change is going online immediately and without any backwards compatability.

CommanderStorm avatar Jul 17 '22 21:07 CommanderStorm

Implemented more information endpoint (nav.tum.sexy/api/get/:id ) and search endpoint (nav.tum.sexy/api/search?q=querry) in https://github.com/TUM-Dev/Campus-Android/pull/1462.

PiotrKedra avatar Jul 18 '22 21:07 PiotrKedra

We have added the localization of the details' endpoint via the ?lang=en query parameter

CommanderStorm avatar Nov 07 '22 18:11 CommanderStorm

Implemented api/search, api/get/{id} and /api/preview/{id} in the UWP TUM Campus-App.

COM8 avatar Jan 09 '23 17:01 COM8

We have merged the /health and the /source_code endpoints into the /status endpoints See this PR for the full discussion. Given that nobody has indicated that he/she is using this API, this is going into production this evening.

CommanderStorm avatar Feb 01 '23 15:02 CommanderStorm

After looking into what https://github.com/TUM-Dev/NavigaTUM/pull/472 needs I have come to the conclusion that this feature would be simpler if we did not have to have two details-responses (one for /api/get/root and one for /api/get/{id}). => /api/get/root is slated for removal in https://github.com/TUM-Dev/NavigaTUM/pull/1154

Given that nobody has indicated that he/she is using this API, this is going into production at the end of the day.

CommanderStorm avatar May 06 '24 00:05 CommanderStorm

poi.nearby_public_transport.mvg was moved into the dedicated /locations/{id}/nearby endpoint. Reason:

  • nobody currently uses that data and it is somewhat large => not worth transmitting for everybody

CommanderStorm avatar Jul 28 '24 12:07 CommanderStorm