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.

Sign up for GitHub

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

Jump to bottom

Basic Navbar design and CSS implementation. #86

Open
wants to merge 32 commits into
base: main
Choose a base branch
from
Open

Basic Navbar design and CSS implementation. #86

wants to merge 32 commits into from

Conversation

MaxAllan-Smith
Copy link

  • Updated the general design on the Navbar while still maintaining the functionality that ed implemented.
  • The logout functionality still needs to be fixed. The page does not refresh properly when user log outs and they can still see 'create new post' functions. However, if the user refreshes the page, the homepage updates correctly.

edwardfgt and others added 30 commits March 7, 2023 12:26
@edwardfgt
create homepage route and basic navbar
e2a4d61
@edwardfgt
add logic for navbar updating when user signsout
80ed053
@forreya
Created CreatePost.js
456eaef
@edwardfgt
add logic for signin, move setToken method and logout method to app.js
3c6d319
@edwardfgt
remove redundant home button
bb20c6c
@edwardfgt
create e2e tests for navbar
c22269b
@edwardfgt
Merge pull request #1 from edwardfgt/create-post
77adfaa 
Created CreatePost.js
@edwardfgt
update test naming
7c8a17c
@edwardfgt
merge new code into navbar branch
7bac5f6
@edwardfgt
revert to c22269b to continue working without errors
8f9e754
@edwardfgt
add comments on reasoning for navbar logic
b68065a
@edwardfgt
update UserSchema to include firstName & lastName
ff88ac6
@edwardfgt
update signup form and post request to include new user information
e4d8c77
@edwardfgt
update all API tests to include new signup fields
eb97fa2
@edwardfgt
update frontend signup tests to include new fields
04e49ff
@edwardfgt
add bootstrap stling to login and signup page
e162453
@edwardfgt
add error message for incorrect login details
1cafc5d
@forreya
Fixed minor bug
254f66d
@edwardfgt
Merge pull request #2 from edwardfgt/create-post
84a0086 
Fixed Minor Bugs in CreatePost.js
@edwardfgt
test storing email in localstorage
96e4321
@edwardfgt
merged from main
0da6b81
@edwardfgt
add createPost and feed components onto the homepage
907e5f1
@edwardfgt
fix CreatePost.js
07ec948
@edwardfgt
feed auto refereshes when new post is made
5188935
@edwardfgt
start creating tests for createPost.js but commented out for the time…
dee5ead 
… being
@edwardfgt
Merge pull request #3 from edwardfgt/navbar
e979744 
Navbar + other changes
@edwardfgt
merge fix with main
511dcaf
@edwardfgt
Merge pull request #4 from edwardfgt/createPostFix
3bc86d0 
Fix createPost component
@MaxAllan-Smith
Branch Update
1f3cbf3
@MaxAllan-Smith
?
89ef879
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@MaxAllan-Smith @edwardfgt @forreya