Skip to content

tr/deploy spike#23

Merged
isTravis merged 16 commits into
mainfrom
tr/deploy-spike
Aug 1, 2023
Merged

tr/deploy spike#23
isTravis merged 16 commits into
mainfrom
tr/deploy-spike

Conversation

@isTravis
Copy link
Copy Markdown
Member

@isTravis isTravis commented Aug 1, 2023

This PR makes some broad changes towards setting the groundwork for non-demo development. I believe none of it should conflict with any of the API work ongoing.

  • Replace Chakra with Tailwind and custom components. See STYLES.md for reasoning behind this decision.
    • I didn't exactly reproduce the layout and styling as it was before (it's noticeably worse now) because there is deeper work to be done to build a more functional UI that used Tailwind's configuration options and capabilities. So, since we're going to re-write these components shortly anyways, I decided to not spend too much time re-implementing the previous style exactly.
  • Update the seed file to create all seed communities at the same time (rather than switching between seed files). This, in combination with a functioning CommunitySwitcher component, allows us to switch between the demo communities, rather than re-running a new seed file each time.
  • Use community slugs for routing given the above update (also, update queries to use this slug for filtering pub/stage/etc results).
  • Add yorkie so that the gitHooks defined in our package.json actually run. Implements prettier on commit, and a type check before pushing. See README.md for details on how to opt out of the latter if needed.

@isTravis isTravis merged commit 94203d3 into main Aug 1, 2023
@isTravis isTravis deleted the tr/deploy-spike branch August 1, 2023 21:27
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