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 support for address autofill in editgrid #574

Merged
merged 2 commits into from
Oct 23, 2023

Conversation

sergei-maertens
Copy link
Member

Closes open-formulieren/open-forms#2656

Simple fix - now the local scope of data is used instead of the global bucket of data.

The 'row' and 'data' value containers are identical when used outside
of a container component, but when inside a container the row provides
the local scope. Edit grid is such a container.
@sergei-maertens sergei-maertens added the needs-backport Fix must be backported to stable release branch label Oct 23, 2023
@codecov
Copy link

codecov bot commented Oct 23, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (156ebad) 71.04% compared to head (9a6ea77) 70.69%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #574      +/-   ##
==========================================
- Coverage   71.04%   70.69%   -0.35%     
==========================================
  Files         209      210       +1     
  Lines        4321     4324       +3     
  Branches     1173     1174       +1     
==========================================
- Hits         3070     3057      -13     
- Misses       1217     1231      +14     
- Partials       34       36       +2     
Files Coverage Δ
src/formio/components/TextField.js 95.23% <100.00%> (+7.14%) ⬆️
src/formio/components/TextField.mocks.js 100.00% <100.00%> (ø)

... and 4 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@sergei-maertens sergei-maertens merged commit 1dc4fa2 into main Oct 23, 2023
11 of 12 checks passed
@sergei-maertens sergei-maertens deleted the issue/2656-address-autofill-in-editgrid branch October 23, 2023 14:53
sergei-maertens added a commit that referenced this pull request Oct 23, 2023
The 'row' and 'data' value containers are identical when used outside
of a container component, but when inside a container the row provides
the local scope. Edit grid is such a container.

Backport-of: #574
sergei-maertens added a commit that referenced this pull request Oct 23, 2023
The 'row' and 'data' value containers are identical when used outside
of a container component, but when inside a container the row provides
the local scope. Edit grid is such a container.

Backport-of: #574
sergei-maertens added a commit that referenced this pull request Oct 23, 2023
The 'row' and 'data' value containers are identical when used outside
of a container component, but when inside a container the row provides
the local scope. Edit grid is such a container.

Backport-of: #574
@sergei-maertens
Copy link
Member Author

Backports: 787c673, dcd5c4b and 292e631

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-backport Fix must be backported to stable release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Get street and city request not triggered from fields in repeating group
2 participants