fix: Edit identity override with prevent flag defaults enabled

[dougfabris]

Thanks for submitting a PR! Please check the boxes below:

• I have added information to docs/ if required so people know about the feature!
• I have filled in the "Changes" section below?
• I have filled in the "How did you test this code" section below?
• I have used a Conventional Commit title for this Pull Request

Changes

It adds the identity verification in hideValue param on the Value component in order to render the input fields, where wasn't possible to edit under identity override

Closes: #4771

How did you test this code?

• Go to project settings
• Enable the prevent flag defaults
• Create a feature
• Go to identities
• Clicking on the feature you created you should be able to edit it

Note to the reviewer: I was looking into it and indeed removing the parameter mentioned to the Value component seems to solve the issue. But I'm afraid I'm missing something because the verification using prevent_flag_defaults was added 2 years ago here #1453

[vercel]

@dougfabris is attempting to deploy a commit to the Flagsmith Team on Vercel.

A member of the Team first needs to authorize it.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
flagsmith-frontend-preview	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 8, 2024 5:56pm

1 Skipped Deployment

Name	Status	Preview	Comments	Updated (UTC)
docs	⬜️ Ignored (Inspect)	Visit Preview		Nov 8, 2024 5:56pm

[github-actions]

Uffizzi Preview deployment-57991 was deleted.

[kyle-ssg]

Hey @dougfabris, thanks for the PR!

Prevent_flag_defaults aims to stop from defining a default remote configuration when creating a flag (which sets a default for all environments), forcing people to set it later per environment. Although your PR does fix an issue where it prevents user overrides, it also stops prevent_flag_defaults from working - we can define on a remote config value on flag creation with this change.

Instead this should do something along the lines of project.prevent_flag_defaults && !identity

[dougfabris]

So in fact I was missing something then. I didn't knew about the remote config you mentioned!
I've applied your suggestion. Thanks for reviewing it @kyle-ssg

[matthewelwell]

Thanks @dougfabris !