-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
[BUG] Fix the dashboard route #85
Comments
You pull the changes?. I work perfect with http://localhost:3000/dashboard/. In my local I am working with port 3001, we should put a configuration at the frontend level to manage env variables to work with the port that everyone wants. Could you try the login? |
yes I tried the changes.
|
I think some of the issues is that the api now requires 3001 and the follow, user and profile controllers still use 3000. we could do PORT=3000
API_PORT=3001 in .env we could do something like this const apiUrl = `http://localhost:${apiPort}/api`; and the same with the app. |
we should have a .env.example for new users too. |
The registration form has not yet uploaded the changes |
The web and api work in same port, remenber that both routes listening for the port of app node. |
I put port 3001 because that's how I have my local, you can change it to 3000, so you can test what I upload a fix for .env variables in the front |
so I am seeing a bunch of changes that need to be made.
there a couple other .env variables I see that I don't have or get, I searched for the _ in the entire codebase. MONGODB_URI=mongodb://username:password@host:port/database We could possibly use something like |
assign it to me |
oh and @jzunigarce the login.schema.js in /schemas needs to be username and password, not email and password |
@gbowne1 we just need a schema for the model. We do not require login, registration scheme, because we do not have such models. We can add the Joi module to validate schema that are submitted by users. The user schema we currently have is for the user collection |
Ok. Good to know that. this reminds me I added these collections for future use: sessions For:
|
The route & path where the dashboard is currently displaying is http://localhost:3000/dashboard/dashboard.html but http://localhost:3000/dashboard/ does not display the dashboard.
The text was updated successfully, but these errors were encountered: