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

Fix question number bug #119

Merged
merged 3 commits into from
Nov 15, 2023
Merged

Conversation

ryanchua00
Copy link
Contributor

No description provided.

@ryanchua00 ryanchua00 self-assigned this Nov 15, 2023
@tobihy
Copy link
Contributor

tobihy commented Nov 15, 2023

LGTM 👍

Copy link
Contributor

@tobihy tobihy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor spelling issue, but LGTM!

README.md Outdated
The api-gateway requires more work, see https://dev.to/naseef012/create-a-microservices-app-with-dockerized-express-api-gateway-1kf9 for more information.
Note that user-service requires a Supabase database and question-service requires a MongoDB database. The respective `README.md` should aid you in creating your own databases.

Also, the ai-service requires your own OpenAi key and email-service requires an SMTP_PASSWORD to run locally. For ease of testing, do so on peerprep.ryanchuahj.com
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OpenAI*

@ryanchua00 ryanchua00 merged commit 9102efe into CS3219-AY2324S1:staging Nov 15, 2023
7 checks passed
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