Rahaf91 / capstone-natural-cures

https://capstone-natural-cures.vercel.app
1 stars 0 forks source link

# User Login #54

Open SylvieDoumet opened 2 months ago

SylvieDoumet commented 2 months ago

Value Proposition

As a user, I want to log into my account securely, In order to access personalized content of the natural cures app.

Description

Image Image

Acceptance criteria

Tasks

SylvieDoumet commented 2 months ago

Rahaf kannst du bitte das Layout dieser User story anpassen - da du sie geschrieben hast würde ich das gerne dir überlassen :-)

F-Kirchhoff commented 2 months ago

Hey Susan, die US schaut super aus, nur der letzte Task ist ein wenig schwammig, was meinst du mit render the component inside component? Zusätzlich solltet ihr mindestens 2 Github OAuth Apps erstellen, eins für eure production Umgebung, also den Main branch eures Projekts und ein zweites, dass auf die einzelnen Review Deployments angepasst werden kann. Die OAuth Apps brauchen nämlich eine eindeutige URL and die die Auth requests gesendet werden können, und die Adresse von dem Review Deployment ändert sich ja mit jedem PR. Deshalb müsst ihr für jeden PR die URL in dieser OAuth App anpassen :/ (und könnt natürlich nur einen aktiven PR zur Zeit haben).

Rahaf91 commented 2 months ago

Thank you for reviewing. Both the login and layout components were enclosed in angle brackets, which is why they weren't displayed...We updated it

rvoll commented 2 months ago

Hi Rahaf & Susan & ..., Looks great - like the rest pf your page! Only the wireframes are a bit sloppy: from the wireframes it is not clear to me what triggers the "Please log in" message. You could make this more obvious. The "Login failed. Please try again." is not included in the wire frames. Anyway, this is a minor issue. Keep it going.

Sannsfilm commented 2 months ago

@rvoll i will updated for the layout, thankyou for review :))