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

Some updates to your readme. #41

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

NuclearMachine
Copy link

Hey, thought your readme could use some help! I know that you mentioned that you will not be accepting any PR's for this repository but these changes could be helpful for newcomers.

I've embedded a video replay for creating a database btw. It's hosted on replayable.io, a company that's sponsoring my contributions.

Let me know what you think.

@whalesink
Copy link

I tried using the GUI tool Navicat to create the database.

image

but when I start it, the backend project (dir: /api) gives me an error accessing the interface. It seems that some tables are missing from the database. Are we missing the step to initialize the database using a script?

@NuclearMachine
Copy link
Author

You're right on that cute anime girl (@whalesink).

According to his readme however:

We're currently using TypeORM's synchronize feature which auto creates the database schema on every application launch.

If I understand this correctly, the database schema should be created on application launch, but I don't see any tables created from my side.

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