fix(client): updated SocialProperty to edit, send and reset data fields

[j-arias-dev]

Description

Updated SocialProperty state to edit, send and reset data fields.

Changes Made

• I removed the useForm in the SocialProperty and updated the fields to point to the state values so that we could track, edit, and submit the updated textField changes.
• Updated the reset functionality by adding a refresh functionality that calls the getLoginProperties reactor.
• Added error handling for modifyLoginProperties.

How to Test

1. Go to the settings page
   Enter admin mode
   Click on configuration
   Select any authentication mode
   Enter value fields for the data
   Click save

2. Expected outcomes

• On the network console, test to see if modifyLoginProperties returns the updated payload values
• For the reset functionality, test to see if the textField values reset to the last submitted value.

Notes

I noticed that we have a description text field for each login property, but I don't see a description value key being returned in the getLoginProperties reactor. Should we remove the description text field?

Also, I disabled the login key text field from being edited, but let me know if we want a user to be able to also edit the login key text field as well.

[github-actions]

@CodiumAI-Agent /describe

[github-actions]

@CodiumAI-Agent /review

[github-actions]

@CodiumAI-Agent /improve

[AAfghahi]

Why not delete this

[AAfghahi]

Is there a reason to leave this commented out

[AAfghahi]

are the reactors fully functional? The reset worked for me, but the save didn't seem to take. Is it waiting for a backend PR to merge?

[QodoAI-Agent]

PR Code Suggestions ✨

Category	Suggestion	Impact
Possible issue	Fix reset API mismatch Align the function’s signature with its usage by either removing the parameter from the call or updating the function to accept an argument. packages/client/src/pages/settings/ConfigurationsPage.tsx [436-438] onClick={() => { - resetLoginProperties(fieldName); + resetLoginProperties(); }} Suggestion importance[1-10]: 8 __ Why: The suggestion correctly identifies a mismatch between the function signature of resetLoginProperties (which takes no parameters) and its usage; updating the call improves consistency and prevents potential runtime errors.	Medium
	Prevent state mutation Avoid direct state mutation by creating a deep clone of the state before modifying it. packages/client/src/pages/settings/ConfigurationsPage.tsx [257-258] -const socialPropsCopy = socialProps; -socialPropsCopy[fieldName][index]['value'] = value; +const socialPropsCopy = { ...socialProps }; +socialPropsCopy[fieldName] = socialPropsCopy[fieldName].map((item, idx) => + idx === index ? { ...item, value } : item +); Suggestion importance[1-10]: 8 __ Why: The suggestion addresses a significant issue by preventing direct state mutation through cloning the socialProps object, which improves code reliability and adheres to best practices in state management.	Medium

[github-actions]

@CodiumAI-Agent /update_changelog

[QodoAI-Agent]

Changelog updates: 🔄

2025-04-10

• Fixed social properties state management and error handling.
• Changed configuration UI by removing redundant fields and logic.

to commit the new content to the CHANGELOG.md file, please type:
'/update_changelog --pr_update_changelog.push_changelog_changes=true'