Skip to content

Updating deprecation notice for projectFingerprint GraphQL

Alana Bellucci requested to merge master-patch-0943 into master

What does this MR do and why?

Action the decision from this discussion: !118553 (comment 1387688698)

Keep projectFingerprint in GraphQL but return uuid from 16.1

Here we have to avoid doing a breaking change outside of a major release (i.e. a stable API output).

My suggestion would be to create a "remove_project_fingerprint_graphql" feature flag. When turned on, projectFingerprint always returns uuid. When turned off, it returns project_fingerprint if it exists, otherwise it returns uuid. This gives users an option to maintain the stability of the output.

Because we only need the existing project_fingerprint values, we can still remove code that generates and uses project_fingerprint, and any remaining code related to it would be in the model and graphql. So we wouldn't use project_fingerprint, for example, for any comparisons internally; the only use for it would be as an external identifier.

Then, in 17.0, we would be able to remove the remaining code and drop the column.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Thiago Figueiró

Merge request reports