building-envelope-data / database

Reference implementation of a product data server as part of the product data network buildingenvelopedata.org
MIT License
0 stars 0 forks source link

The GraphQL field `id` of `Node`s should also encode the locale #26

Open simon-wacker opened 2 years ago

simon-wacker commented 2 years ago

For the reasons see https://github.com/building-envelope-data/api/blob/f75edaac57e325092effceca10177ae11d2c3164/apis/database.graphql#L1-L10 and https://github.com/building-envelope-data/api/blob/f75edaac57e325092effceca10177ae11d2c3164/apis/database.graphql#L417-L440 Currently, it encodes only the UUID (together with the table name to be unique in the database), see https://github.com/building-envelope-data/database/blob/d45f78d150b58e016d63a827124b740765c4d350/backend/src/GraphQl/Entities/EntityType.cs#L17-L24, and thus violates field stability. The same is true for the metabase project.