Skip to content

fix: toggling settings section#2349

Merged
miguelpeixe merged 1 commit intomasterfrom
fix/settings-section-toggling
Mar 16, 2023
Merged

fix: toggling settings section#2349
miguelpeixe merged 1 commit intomasterfrom
fix/settings-section-toggling

Conversation

@miguelpeixe
Copy link
Member

All Submissions:

Changes proposed in this Pull Request:

#2259 introduced a bug due to this change. The API results not being updated back to the component, the activation toggle doesn't change update on success.

This PR restores the update but preserves the original intention of the removal, which is to not let a section update affect the values of other rendered sections.

Closes #2336

How to test the changes in this Pull Request:

Confirm #2336 is no longer reproducible.

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes, as applicable?
  • Have you successfully ran tests with your changes locally?

@miguelpeixe miguelpeixe added the [Status] Needs Review The issue or pull request needs to be reviewed label Mar 15, 2023
@miguelpeixe miguelpeixe requested a review from a team as a code owner March 15, 2023 17:00
@miguelpeixe miguelpeixe self-assigned this Mar 15, 2023
@github-actions github-actions bot added [Status] Approved The pull request has been reviewed and is ready to merge and removed [Status] Needs Review The issue or pull request needs to be reviewed labels Mar 16, 2023
@miguelpeixe miguelpeixe merged commit 87d5218 into master Mar 16, 2023
@miguelpeixe miguelpeixe deleted the fix/settings-section-toggling branch March 16, 2023 16:45
matticbot pushed a commit that referenced this pull request Mar 16, 2023
# [1.108.0-alpha.1](v1.107.1...v1.108.0-alpha.1) (2023-03-16)

### Bug Fixes

* **ads:** proper usage of useEffect cleanup function ([#2335](#2335)) ([58e4c10](58e4c10))
* **cli:** setup config path ([6de5d36](6de5d36))
* **data-events:** mailchimp metadata keys ([#2331](#2331)) ([de82e06](de82e06))
* **modal-checkout:** provide conversion URL for ESP ([feca20d](feca20d))
* only show perfmatters notice if defaults are used ([bdae0b2](bdae0b2))
* post-release merge conflict ([3d03cd2](3d03cd2))
* show handoff to finish Newspack setup only if setup is incomplete ([#2343](#2343)) ([b0a85ef](b0a85ef))
* toggling settings section ([#2349](#2349)) ([87d5218](87d5218))
* **webhooks:** endpoint requests modal label ([#2332](#2332)) ([fd99ff2](fd99ff2))
* **webhooks:** grid layout ([4aa898f](4aa898f))
* **wizards:** broken rendering in WP 6.2 ([a6cef5a](a6cef5a))

### Features

* add GA4 donation events ([#2326](#2326)) ([a1377a2](a1377a2))
* Add GA4 Newsletters subscriptions events ([#2330](#2330)) ([86ffc66](86ffc66))
* Add popups donation events ([#2310](#2310)) ([53e0c4a](53e0c4a))
* **amp-deprecation:** polyfills for amp-img, amp-iframe, amp-fit-text, amp-youtube ([#2308](#2308)) ([7419cd5](7419cd5))
* **health-check:** mark some plugins as supported ([#2329](#2329)) ([c1774e3](c1774e3))
@matticbot
Copy link
Contributor

🎉 This PR is included in version 1.108.0-alpha.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

matticbot pushed a commit that referenced this pull request Mar 20, 2023
# [1.109.0-alpha.1](v1.108.0...v1.109.0-alpha.1) (2023-03-20)

### Bug Fixes

* **ads:** proper usage of useEffect cleanup function ([#2335](#2335)) ([58e4c10](58e4c10))
* **cli:** setup config path ([6de5d36](6de5d36))
* **data-events:** mailchimp metadata keys ([#2331](#2331)) ([de82e06](de82e06))
* **modal-checkout:** provide conversion URL for ESP ([feca20d](feca20d))
* only show perfmatters notice if defaults are used ([bdae0b2](bdae0b2))
* post-release merge conflict ([3d03cd2](3d03cd2))
* show handoff to finish Newspack setup only if setup is incomplete ([#2343](#2343)) ([b0a85ef](b0a85ef))
* toggling settings section ([#2349](#2349)) ([87d5218](87d5218))
* **webhooks:** endpoint requests modal label ([#2332](#2332)) ([fd99ff2](fd99ff2))
* **webhooks:** grid layout ([4aa898f](4aa898f))
* **wizards:** broken rendering in WP 6.2 ([a6cef5a](a6cef5a))

### Features

* add GA4 donation events ([#2326](#2326)) ([a1377a2](a1377a2))
* Add GA4 Newsletters subscriptions events ([#2330](#2330)) ([86ffc66](86ffc66))
* Add popups donation events ([#2310](#2310)) ([53e0c4a](53e0c4a))
* **amp-deprecation:** polyfills for amp-img, amp-iframe, amp-fit-text, amp-youtube ([#2308](#2308)) ([7419cd5](7419cd5))
* **health-check:** mark some plugins as supported ([#2329](#2329)) ([c1774e3](c1774e3))
@matticbot
Copy link
Contributor

🎉 This PR is included in version 1.109.0-alpha.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

matticbot pushed a commit that referenced this pull request Mar 24, 2023
# [1.110.0-alpha.1](v1.109.0...v1.110.0-alpha.1) (2023-03-24)

### Bug Fixes

* **ads:** proper usage of useEffect cleanup function ([#2335](#2335)) ([58e4c10](58e4c10))
* **cli:** setup config path ([6de5d36](6de5d36))
* **data-events:** mailchimp metadata keys ([#2331](#2331)) ([de82e06](de82e06))
* **modal-checkout:** provide conversion URL for ESP ([feca20d](feca20d))
* only show perfmatters notice if defaults are used ([bdae0b2](bdae0b2))
* **perfmatters:** add twitter.com to JS delay list ([34bac52](34bac52))
* post-release merge conflict ([3d03cd2](3d03cd2))
* show handoff to finish Newspack setup only if setup is incomplete ([#2343](#2343)) ([b0a85ef](b0a85ef))
* toggling settings section ([#2349](#2349)) ([87d5218](87d5218))
* **webhooks:** endpoint requests modal label ([#2332](#2332)) ([fd99ff2](fd99ff2))
* **webhooks:** grid layout ([4aa898f](4aa898f))
* **wizards:** broken rendering in WP 6.2 ([a6cef5a](a6cef5a))

### Features

* add GA4 donation events ([#2326](#2326)) ([a1377a2](a1377a2))
* Add GA4 Newsletters subscriptions events ([#2330](#2330)) ([86ffc66](86ffc66))
* Add popup events to GA4 ([#2337](#2337)) ([62c738e](62c738e))
* Add popups donation events ([#2310](#2310)) ([53e0c4a](53e0c4a))
* **amp-deprecation:** polyfills for amp-img, amp-iframe, amp-fit-text, amp-youtube ([#2308](#2308)) ([7419cd5](7419cd5))
* **health-check:** mark some plugins as supported ([#2329](#2329)) ([c1774e3](c1774e3))
@matticbot
Copy link
Contributor

🎉 This PR is included in version 1.110.0-alpha.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

matticbot pushed a commit that referenced this pull request Mar 27, 2023
# [1.110.0](v1.109.0...v1.110.0) (2023-03-27)

### Bug Fixes

* **ads:** proper usage of useEffect cleanup function ([#2335](#2335)) ([58e4c10](58e4c10))
* **cli:** setup config path ([6de5d36](6de5d36))
* **data-events:** mailchimp metadata keys ([#2331](#2331)) ([de82e06](de82e06))
* **modal-checkout:** provide conversion URL for ESP ([feca20d](feca20d))
* only show perfmatters notice if defaults are used ([bdae0b2](bdae0b2))
* **perfmatters:** add twitter.com to JS delay list ([34bac52](34bac52))
* post-release merge conflict ([3d03cd2](3d03cd2))
* show handoff to finish Newspack setup only if setup is incomplete ([#2343](#2343)) ([b0a85ef](b0a85ef))
* toggling settings section ([#2349](#2349)) ([87d5218](87d5218))
* **webhooks:** endpoint requests modal label ([#2332](#2332)) ([fd99ff2](fd99ff2))
* **webhooks:** grid layout ([4aa898f](4aa898f))
* **wizards:** broken rendering in WP 6.2 ([a6cef5a](a6cef5a))

### Features

* add GA4 donation events ([#2326](#2326)) ([a1377a2](a1377a2))
* Add GA4 Newsletters subscriptions events ([#2330](#2330)) ([86ffc66](86ffc66))
* Add popup events to GA4 ([#2337](#2337)) ([62c738e](62c738e))
* Add popups donation events ([#2310](#2310)) ([53e0c4a](53e0c4a))
* **amp-deprecation:** polyfills for amp-img, amp-iframe, amp-fit-text, amp-youtube ([#2308](#2308)) ([7419cd5](7419cd5))
* **health-check:** mark some plugins as supported ([#2329](#2329)) ([c1774e3](c1774e3))
@matticbot
Copy link
Contributor

🎉 This PR is included in version 1.110.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released on @alpha released [Status] Approved The pull request has been reviewed and is ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Toggling a wizard setting section doesn't update the UI

3 participants