EnCiv / undebate-ssp

Undebate Self Service Portal- Web portal where election administrators of democratically run organizations create undebates for their candidates and voters
Other
6 stars 9 forks source link

Various Defects/Issues #162

Closed ice1080 closed 2 years ago

ice1080 commented 2 years ago

This issue is being used to hold various issues noticed at any time in any part of the app or storybook. It can be added to at any time prior to the v1 release

ddfridley commented 2 years ago

I fixed Vertical Timeline story - it was an issue with uninitialized data in time-point.js It will be pushed in an upcoming PR.

ice1080 commented 2 years ago

@ddfridley any preference on the method renames? I'm fine to leave it if you prefer or can do refactoring to rename methods to be more descriptive if preferred. I'm specifically referring to these two:

But I was also thinking of renaming multiple things where the only difference between moderator and candidate is singular vs plural.

ddfridley commented 2 years ago

It would be much easier to read if it was ...ModeratorSubmission and CandidateSubmissions.

Another challenge: If you look at app/components/done-locked-button * have created a button that the user has to click to say that they're done with that panel. But I haven't factored that additional status info in to a lot of places. Frankly, it's not immediately clear where all the places are that it needs to be done.

I know several people have been contributing to this file. Is there a way to do the naming to make it more apparent all the places that need a change if something new is added to a panel. (I added some new fields like organizationUrl and organizationLogo).

I don't know a good way to do this. maybe just fixing the names will be enough but I'd welcome your thoughts.

Thanks, david