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

Init user service #39

Merged
merged 13 commits into from
Sep 28, 2024
Merged

Init user service #39

merged 13 commits into from
Sep 28, 2024

Conversation

LimZiJia
Copy link

What is in this PR

This is just the basic user-service. It is dockerised, can connect to an atlas cluster and the endpoints are working

What is not in this PR

Documentation, tests and connection to the login page.

Checklist

  • I have updated documentation
  • All tests passing

Connecting to server

This user-service is run on my own free atlas mongodb server. To connect to db, make an atlas cluster. It will show on the dashboard like this:
image

Click on connect > drivers > 3. Add your connection string into your application code

image

Copy paste as USER_SERVICE_CLOUD_URI in the .env file with the password you set earlier.

samuelim01 and others added 13 commits September 27, 2024 20:19
Got user service to work in docker. It can only interface with an atlas cloud server currently.
User-service in docker can only communicate with an atlas mongodb server.
Some files should have been committed in the previous commit. They are
mostly to do with the dockerisation process. The merge with main might
have messed with it.
Somehow .gitignore was not commited with the last commit
Apparently bcrypt library cannot run on Docker's architecture. Change
bcrypt to bcryptjs for compatibility.
Test does not currently work
Not connected to the login interface yet. Commiting to make a clean Pr.
* Remove orphan history file
* Move `index.js` and `server.js` to the correct directory
* Reduce use of port
Copy link

@samuelim01 samuelim01 left a comment

Choose a reason for hiding this comment

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

LGTM

@samuelim01 samuelim01 merged commit c1861bb into main Sep 28, 2024
2 checks passed
@samuelim01 samuelim01 mentioned this pull request Sep 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants