search

digitalfabrik / integreat-cms

Simplified content management back end for the Integreat App - a multilingual information platform for newcomers
https://digitalfabrik.github.io/integreat-cms/
Apache License 2.0
56 stars 33 forks source link

News have stringified id #2927

Closed steffenkleinle closed 1 month ago

steffenkleinle commented 1 month ago

Describe the Bug

The ids of news are returned as strings instead of numbers.

Steps to Reproduce

  1. Go to https://cms-test.integreat-app.de/api/v3/testumgebung/de/fcm/
  2. See stringified id

Expected Behavior

The id to be of type number.

Actual Behavior

The id is a string.

Additional Information

This is also in contrast to all other endpoints using numbers. Also the ids in the available languages are numbers (#2811).

MizukiTemma commented 1 month ago

@steffenkleinle Is the id currently being used and do we need compatibility?

steffenkleinle commented 1 month ago

@steffenkleinle Is the id currently being used and do we need compatibility?

The id is currently used, yes. However, I think we do not need compatibility (as we were already expecting a numeric id/using typecasts so far).

MizukiTemma commented 1 month ago

@steffenkleinle Is the id currently being used and do we need compatibility?

The id is currently used, yes. However, I think we do not need compatibility (as we were already expecting a numeric id/using typecasts so far).

@steffenkleinle Ok, thank you 😸