Skip to content

[#4] [FRONTEND] As a user, I can sign-up using my email and password#28

Merged
longnd merged 16 commits intodevelopfrom
feature/sign-up-front-end
Apr 20, 2021
Merged

[#4] [FRONTEND] As a user, I can sign-up using my email and password#28
longnd merged 16 commits intodevelopfrom
feature/sign-up-front-end

Conversation

@longnd
Copy link
Owner

@longnd longnd commented Apr 14, 2021

Resolves #4

What happened

  • This PR creates the front-end for the sign-up feature.
  • I also rename the project from "gscraper_web" -> "gscraper" because the former name creates a confusion because Phoenix also create 2 directories inside the project (lib/gcraper_web & lib/gcraper_web_web)

Insight

  • Add Boostrap
  • Create a route for signing up
  • Style the page

Proof Of Work

Screen Shot 2021-04-14 at 16 37 01

@longnd longnd added this to the 1.0.0 milestone Apr 14, 2021
@longnd longnd self-assigned this Apr 14, 2021
@longnd longnd force-pushed the feature/sign-up-front-end branch from 951f557 to 57a9eaf Compare April 16, 2021 04:27
@longnd longnd force-pushed the feature/sign-up-front-end branch from 57a9eaf to 0bf017a Compare April 16, 2021 04:29
Copy link
Collaborator

@hanam1ni hanam1ni left a comment

Choose a reason for hiding this comment

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

A few comments and it would be easier for the next reviewer if we can separate the rename changes to another PR.

Copy link
Collaborator

@rosle rosle left a comment

Choose a reason for hiding this comment

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

Other than Micky's comment. Looking good to me 👍 ✅

Copy link
Collaborator

@olivierobert olivierobert left a comment

Choose a reason for hiding this comment

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

Ideally, the renaming of the project and set up of Bootstrap should be have been done in separate PRs to avoid having a PR with 80+ file changes 🙈

Copy link
Collaborator

@hanam1ni hanam1ni left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@longnd longnd merged commit 41b4afb into develop Apr 20, 2021
@longnd longnd deleted the feature/sign-up-front-end branch April 20, 2021 03:06
@longnd longnd mentioned this pull request Sep 21, 2021
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.

[FRONTEND] As a user, I can sign-up using my email and password

4 participants