-
Notifications
You must be signed in to change notification settings - Fork 446
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
Arnau's auth app #342
base: master
Are you sure you want to change the base?
Arnau's auth app #342
Conversation
… homepage, footer... and add styling to them
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey Arnau, hahah love Pablo Escobar XD did you test your frontend using your auth API? I can't register a user,I'm afraid the logic are swapped: what is the signup page now should be login and viceversa. Please double check the FE app and endpoints are working, thanks :)
app.get("/", (req, res) => { | ||
res.send("Hello Technigo!"); | ||
const endpoints = expressListEndpoints(app); | ||
const documentation = { | ||
Welcome: "This is the Authentication API!", | ||
Endpoints: { | ||
"/": "Get API documentation", | ||
"/users": { | ||
POST: "Create a new user", | ||
}, | ||
"/sessions": { | ||
POST: "Authenticate a returning user", | ||
}, | ||
"/secrets": { | ||
GET: "Get secret content (requires authentication)", | ||
}, | ||
}, | ||
}; | ||
res.json(documentation); | ||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you're not really listing the endpoints urls, try with:
app.get("/", (req, res) => {
const endpoints = expressListEndpoints(app);
const documentation = {
Welcome: "This is the Authentication API!",
Endpoints: endpoints.map((endpoint) => ({
method: endpoint.methods.join(", "),
path: endpoint.path
}))
};
res.json(documentation);
});
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry Antonella, I forgot to tell that I couldn't finish the auth project. I got stuck first on getting the endpoints to work on postman and later trying to deploy to Netlify, so I had no time to connect the FE to the BE. I will try to work on it in another moment. Just now I'm working on the FP.
return ( | ||
<div className="button-container"> | ||
<button onClick={onClick} type="button"> | ||
Sign In |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sign in and Login are synonyms, it's sign up or register if you prefer :)
Hello Arnau, don't forget to update this project please 👍 |
Netlify link
backend: here
frontend: here
Solo