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

I26 create a home page #91

Merged
merged 105 commits into from Aug 15, 2022
Merged

I26 create a home page #91

merged 105 commits into from Aug 15, 2022

Conversation

JadeynFeng
Copy link
Collaborator

@JadeynFeng JadeynFeng commented Jul 20, 2022 •
edited

Change Summary

added the home page, with summary dashboard and a new visit form.

Change Form

Fill this up (NA if not available). If a certain criteria is not met, can you please give a reason.

  • [ issue 26 and 33 (the backend for homepage just needs to be done in a different issue)] The pull request title has an issue number
  • [ yup] The change works by "Smoke testing" or quick testing
  • [nope] The change has tests
  • [nope, but comments in code for modal.tsx] The change has documentation

Other Information

[Is there anything in particular in the review that I should be aware of?]
Just that the backend is yet to be completed, and will be done in a separate issue.

Related Issue

chayG5 and others added 30 commits July 2, 2022 14:03
@chayG5
basic files
9274057
@LisonLau @chayG5
home page layout and top nav
d5a7c1e 
added top nav
added summary dashboard
added start/stop button

Co-Authored-By: Chayanika Gangopadhyay <104259880+chayG5@users.noreply.github.com>
@LisonLau
minor changes
fcc12db
@chayG5
fixed red line, better quality logo and centre elements of summary da…
5faaa9f 
…shboard
@chayG5
summary dashboard and button fixed/centred
86c100c
@LisonLau
test commit
a9da7d8
@LisonLau
change font size, added modal
9767375
@chayG5
when click on start or stop update form. Also, a dog picture when vis…
24f1bb7 
…it hasn't started
@chayG5
react-hook-form
e657016
@chayG5
adding type of visit and distance covered
18b1529
@chayG5
multi select drop down, installed react select
2ee3d36
@chayG5
fix drop down
f6be041
@LisonLau
update modal forms
20226c5
@LisonLau
formatting changes
fa19620
@LisonLau
formatting changes
616b38d
@LisonLau
firebase document init
ab0c097
@LisonLau
up to date with main
3f27c02
@chayG5
remove commented code
a9e333e
@LisonLau
added labels for visit form
69b302d
@LisonLau
added colour, changed position of button
77680ca
@LisonLau
making font smaller
679ef01
@LisonLau
separated form selectors into components, fixed type error
d8b976f
@LisonLau
up to date with main
beb5187
@LisonLau
fixed top nav
4991ed3
@LisonLau
added new form fields, changed font size, modal size, etc.
1b40bf8
@JadeynFeng
added dog image and positioning
f138869
@JadeynFeng
Merge branch 'i26-create_a_home_page' of https://github.com/codersfor…
da86837 
…causes/poops into i26-create_a_home_page
@LisonLau
Added form field component
110f54b
@chayG5
fix visit form width
a04c3b1
@chayG5
stop button functionality and cancel button
6582786
@LisonLau LisonLau linked an issue Jul 30, 2022 that may be closed by this pull request
Add a start/stop visit button in the Home page #33
Closed
4 tasks
@NicholasChoong NicholasChoong merged commit ae42847 into main Aug 15, 2022
@NicholasChoong NicholasChoong deleted the i26-create_a_home_page branch August 15, 2022 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LisonLau LisonLau LisonLau left review comments

@chayG5 chayG5 chayG5 left review comments

@NicholasChoong NicholasChoong NicholasChoong approved these changes

@skyheat skyheat Awaiting requested review from skyheat

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add a start/stop visit button in the Home page Create a Home page
5 participants
@JadeynFeng @LisonLau @skyheat @NicholasChoong @chayG5