add plans page in naaar

[gmrabian]

Description

Add the "Add Plans" page to the naaar report. Copied from mcpar with some verbiage and metadata updates.

Notes/Questions: Do we need to add plans to the entities object? I did but I'm not exactly sure where/how it is used

Related ticket(s)

CMDCT-4098

---

How to test

• Run app or view deployed branch
• Log in as any state user
• Create a NAAAR
• Enter the NAAAR and verify there is an add plans page
• Verify you can add and remove plans
• Verify it matches the figma design

---

Pre-review checklist

• [ ] I have added thorough tests, if necessary
• [ ] I have updated relevant documentation, if necessary
• I have performed a self-review of my code
• I have manually tested this PR in the deployed cloud environment

---

Pre-merge checklist

Review

• Design: This work has been reviewed and approved by design, if necessary
• Product: This work has been reviewed and approved by product owner, if necessary

---

[bangbay-bluetiger]

Tested and worked for me.

[ntsummers1]

Works great for me! Tested in the deployed instance.

Nit:
I've always hated this pages spacing between the dynamic fields and text describing the page. What if we tightened this up a little?
Could overwrite the dynamicField and textField css props we have in the DynamicField.tsx file with something like this:

dynamicField: {
    alignItems: "flex-end",
    marginTop: "1.5rem",
    ".desktop &": {
      width: "32rem",
    },
    ".tablet &": {
      width: "29rem",
    },
    ".ds-u-clearfix": {
      width: "100%",
    },
  },
  textField: {
    width: "100%",

    label: {
      marginTop: "0",
    },
  },

The key things I changed being the marginTop gets moved to the dynamic field css and we overwrite the label in the textfield from having the marginTop by making it 0.

[karla-vm]

re: your question, I believe plans are going to be an entity going forward, so I would keep it as part of this PR. We'll be needing to have some discussions around the data structure of NAAAR soon 💯

[gmrabian]

@ntsummers1 I added the changes and noticed the marginTop in dynamicField didn't affect anything so I just added the label prop

[codeclimate]

Code Climate has analyzed commit 8d1a844 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (90% is the threshold).

This pull request will bring the total coverage in the repository to 96.7% (0.0% change).

View more on Code Climate.