Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add stories for UT-HAI's client-deliberation fork of client-participation #7

Merged
merged 36 commits into from
Oct 12, 2024

Conversation

patcon
Copy link
Member

@patcon patcon commented Oct 6, 2024

Makes progress on #5

Screen.Recording.2024-10-06.at.2.17.35.PM.mov
  • added the ones that were simple to bring into storybook, and only from the interface previously known as client-participation (client-admin pages are also part of client-deliberation, but I presume those are mostly just theme-ui changes in color scheme...?)

Note: While it's much clearer than upstream, there is still a few sporadic uses of the "PolisNet" library to make non-mockable API calls directly in component logic. This is maybe an opportunity for improvement.

I'm going to leave this as a PR for a few days, and also try to get #6 working in the meantime, so that it's easier to review online

cc: @jcslynn @jakerm2002 @JanSteinhauer (UT-HAI repo contributors)
ps, thank you SO MUCH for helping to push the project forward with lots of this public work!

@patcon
Copy link
Member Author

patcon commented Oct 6, 2024

Rockin. This is now auto-deploying, and exists at:
https://civictechto.github.io/polis-storybook/PR-7/

@patcon patcon mentioned this pull request Oct 6, 2024
6 tasks
@patcon
Copy link
Member Author

patcon commented Oct 12, 2024

Going to merge this as-is, since it gives a sense of what's going on in the fork, and how many components they're migrated to React. Remaining todays can be tracked in other issue.

@patcon patcon merged commit 3cc54bd into main Oct 12, 2024
2 checks passed
@patcon patcon deleted the add-UT-HAI-client-deliberation branch October 12, 2024 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant