Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for editing settings with simple form and rails 7.1 (Koi 2.6) #584

Merged
merged 1 commit into from
Mar 7, 2024

Conversation

sobakasu
Copy link
Contributor

@sobakasu sobakasu commented Mar 7, 2024

related to heartcombo/simple_form#1834 The previous html for editing a setting looked like this:

<input class="string optional form--medium enhanced" type="text" name="setting[setting[36086]][value]" id="setting_setting_70243__value">

it now looks like:

<input value="Job position" class="string required form--medium enhanced" type="text" name="setting[setting_36086][value]" id="setting_setting_36086_value">

@sobakasu sobakasu changed the title Fix for editing settings with simple form and rails 7.1 Fix for editing settings with simple form and rails 7.1 (Koi 2.6) Mar 7, 2024
related to heartcombo/simple_form#1834
The previous html for editing a setting looked like this:

<input class="string optional form--medium enhanced" type="text" name="setting[setting[36086]][value]" id="setting_setting_70243__value">

it now looks like:

<input value="Job position" class="string required form--medium enhanced" type="text" name="setting[setting_36086][value]" id="setting_setting_36086_value">
Copy link

@brendanh-katalyst brendanh-katalyst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nested squarebrackets shaking my head

@sobakasu sobakasu merged commit 0c65391 into release/2.6 Mar 7, 2024
1 check passed
@sobakasu sobakasu deleted the fix/simple-form-settings branch March 7, 2024 04:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants