The SoilDisturbance component is typically nested inside of a BAccordion element when used in a page (e.g. Direct Seeding, Transplanting). Currently the BAccordion element is in the pages. It should be moved into the component so that it can just be a part of the component any time it is used.
This should also include adding a prop (includeAccordion, default:true) to the SoilDisturbance component that optionally omits the BAccordion so that the component can be used without the accordion if desired (e.g. in a Soil Disturbance form).
To do:
[ ] - extend SoilDisturbance to include BAccordion
[ ] - update SoilDisturbance tests accordingly
[ ] - refactor Direct Seeding form and its associated tests.
[ ] - refactor Transplanting form and its associated tests.
I no longer think this is the correct approach. This would required duplication of all of the code inside the accordion so that it could be displayed or not.
The
SoilDisturbance
component is typically nested inside of aBAccordion
element when used in a page (e.g. Direct Seeding, Transplanting). Currently theBAccordion
element is in the pages. It should be moved into the component so that it can just be a part of the component any time it is used.This should also include adding a prop (
includeAccordion
, default:true
) to theSoilDisturbance
component that optionally omits theBAccordion
so that the component can be used without the accordion if desired (e.g. in a Soil Disturbance form).To do:
SoilDisturbance
to includeBAccordion
SoilDisturbance
tests accordingly