Keeps current Accordion sections open

[supernova-at]

Description

This PR fixes the problem where Accordion sections were "resetting" back to their initial state after mutations occurred in subsections (add a coupon, apply a gift card, etc.).

This was because the useEffect function that enforced the initial setting of which sections should be open or closed was firing whenever Accordion children changed, which includes the contents of the Accordion sections. And these sections children were often changing based on the results of a mutation. For example: a list of gift cards would appear once you apply a gift card.

This PR also sets the price adjustments accordion to have all sections closed initially.

Related Issue

Closes PWA-359.

Acceptance

Verification Stakeholders

Specification

Verification Steps

1. Go to the /cart page
2. Verify that the price adjustments accordion initially doesn't have any sections open
3. Open the Coupon or GiftCard section
4. Apply or remove a coupon or gift card
5. See that the section remains open and that the Accordion doesn't reset back to no sections open.

Screenshots / Screen Captures (if appropriate)

Checklist

• I have updated the documentation accordingly, if necessary.
• I have added tests to cover my changes, if necessary.

[PWAStudioBot]

	Messages
📖	Access a deployed version of this PR here. Make sure to wait for the "pwa-pull-request-deploy" job to complete.
📖	DangerCI Failures related to missing labels/description/linked issues/etc will persist until the next push or next nightly build run (assuming they are fixed).
📖	Associated JIRA tickets: PWA-359.

Generated by 🚫 dangerJS against 2d3e33d

[sirugh]

This PR also sets the price adjustments accordion to have all sections closed initially.

@tjwiebell just told me that UX says to keep the shipping methods open initially.

[tjwiebell]

This PR also sets the price adjustments accordion to have all sections closed initially.

@tjwiebell just told me that UX says to keep the shipping methods open initially.

Until this bug is merged; latest mocks have all sections collapsed on first render.

[schensley]

UX approved

[supernova-at]

PR Updated:

• Fixed merge conflicts

[dpatil-magento]

Approved latest commits.