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

chore(easier-setup): Easier start for new developers #53

Merged
merged 2 commits into from
Oct 10, 2024

Conversation

JumpLink
Copy link
Collaborator

@JumpLink JumpLink commented Oct 8, 2024

To make it easier for the new developer to get started, I have continued to work on the developer experience:

  • Easy startup, where everything needed to start can be done in the root directory.
  • The docker-compose.yml and the corresponding .env are now in the root directory.
  • vite and tsc no longer clear the screen in watch mode, which previously caused important information to be lost in the console.
  • To be able to disable the clearScreen option I also updated vite and the plugins used.
  • README.md updated for an easy start.
  • Added information to the backend`s README.md on how to make a test purchase with a test credit card.
  • I have also added a few new scripts to the package.json, for example presetup which should be executed before the first start of the container stack and setup which should be executed when the container stack is running.

@JumpLink JumpLink requested a review from wpf500 October 8, 2024 14:22
@wpf500
Copy link
Member

wpf500 commented Oct 10, 2024

This is great. As a next step (in a separate PR), how about resolving #1 and moving the router into this repo? Then the docker-compose file can have it as a service and we can delete all the proxy config from the frontend.

@JumpLink
Copy link
Collaborator Author

Yea, would like to do this 👍

@JumpLink JumpLink merged commit e30d617 into main Oct 10, 2024
7 checks passed
@wpf500 wpf500 deleted the chore/easier-start branch October 30, 2024 15:57
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.

2 participants