Updating deprecation notice for projectFingerprint GraphQL
What does this MR do and why?
Action the decision from this discussion: !118553 (comment 1387688698)
projectFingerprint
in GraphQL but return uuid
from 16.1
Keep 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.
-
I have evaluated the MR acceptance checklist for this MR.