StateVoicesNational / Spoke

mass-contact text/SMS distribution tool
Other
463 stars 403 forks source link

New node 20 branch texter side boxes #2360

Closed engelhartrueben closed 6 months ago

engelhartrueben commented 6 months ago

Fixes # (issue)

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change, and any blockers that make your change a WIP

Mostly QOL, adding comments and what not.

The ctrl+click text is new. I didn't know I could do this, which probably means others don't as well image

But the GSIntegerField.jsx was necessary due to the stricter GraphQL update that just occurred. Previously, options looking to gather user input for numbers, like batch size, would use GSFormField, which would return a string that contained a number. Now, with GSIntegerField, the user input can be passed directly as an integer.

Checklist:

NOTE

I have not looked at merge conflicts - just getting this out here before class. Will be able to come back at 4/3/2024 4:30 EST to look that over.

engelhartrueben commented 6 months ago

There is a cypress test failing, which is odd. Will need more time to look into it prior to merging

engelhartrueben commented 6 months ago

Cypress Test resolved.