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.

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

combined signup and profile #147

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

combined signup and profile #147

wants to merge 15 commits into from

Conversation

JSutanto19
Copy link
Collaborator

Fixes #

How to Test
Replace this with instructions on how to minimally test that this PR is working.

Change Summary
Replace this with a sentence describing what this PR does for customer users, or N/A if customers aren't affected.

Developer Checklist

  • This PR includes appropriate automated tests for touched features.
  • Thorough manual testing was done.
  • Code was commented and documentation was updated with the change.

@github-actions
Copy link

Total Coverage: 37.39%

Coverage Report
FileBranchesFuncsLinesUncovered Lines
src/components
   CartView.js100%88.89%95.83%77
   DashboardDescriptionCard.js10%30%36.36%106, 107, 119, 133, 286, 42, 43, 43, 46, 46, 46, 47, 48, 49, 50, 51, 52, 53, 62, 64, 64, 64, 69, 84, 94, 95, 98, 98, 98, 99
   DashboardMessages.js55.26%48%73.24%100, 154, 157, 176, 180, 186, 190, 196, 197, 215, 215, 215, 216, 222, 242, 242, 242, 243, 249, 269, 269, 269, 270, 276, 300, 300, 300, 301, 307, 318, 327, 365, 375, 399, 47, 57
   DashboardOpenHourCard.js40%45.45%51.43%102, 116, 133, 137, 214, 37, 38, 38, 41, 41, 41, 42, 43, 44, 52, 53, 55, 55, 55, 60, 68, 74, 75, 78, 78, 78, 79, 90, 91
   DashboardView.js25%40.91%36.11%109, 110, 112, 113, 120, 134, 135, 137, 138, 143, 158, 178, 179, 179, 179, 180, 181, 182, 185, 295, 57, 89, 91, 93, 94, 99
   DashboardViewAllMessage.js100%0%0%10, 12, 15
   FoodItemCard.js59.38%59.26%50.75%100, 100, 100, 101, 105, 107, 114, 123, 124, 126, 126, 126, 129, 138, 139, 141, 141, 141, 142, 146, 147, 151, 154, 154, 154, 155, 166, 168, 174, 182, 184, 185, 193, 311, 329, 466, 467, 474, 79, 80, 81, 82, 98, 98, 98, 99
   FoodSearchView.js33.33%50%47.73%101, 102, 104, 121, 125, 126, 133, 134, 151, 55, 56, 57, 67, 71, 71, 71, 72, 73, 75, 76, 85, 86, 87, 88, 89, 90, 92
   HomeView.js100%0%0%12, 16
   InventoryView.js66.67%78.95%83.33%103, 108, 110, 115, 147, 56, 57, 58
   ListView.js0%0%0%100, 100, 100, 100, 100, 102, 102, 102, 103, 103, 103, 104, 106, 108, 108, 108, 109, 109, 109, 110, 112, 116, 120, 125, 126, 126, 126, 126, 126, 127, 128, 157, 160, 167, 183, 189, 193, 25, 26, 30, 31, 35, 39, 39, 39, 39, 39, 40, 42, 43, 46, 46, 46, 48, 49, 53, 57, 59, 59, 59, 62, 64, 65, 66, 68, 75, 76, 77, 79, 80, 81, 82, 83, 83, 83, 84, 85, 86, 89, 90, 91, 93, 94, 95, 97, 97, 97, 97, 97, 99
   Map.js0%0%0%101, 103, 103, 103, 104, 104, 104, 105, 107, 111, 115, 119, 120, 121, 128, 135, 136, 137, 141, 148, 156, 160, 161, 162, 169, 169, 169, 170, 173, 177, 178, 188, 190, 20, 201, 205, 205, 217, 217, 22, 28, 35, 36, 49, 50, 51, 52, 53, 54, 56, 60, 61, 63, 70, 71, 72, 74, 75, 76, 77, 78, 78, 78, 79, 80, 81, 84, 85, 86, 88, 89, 90, 92, 92, 92, 92, 92, 94, 95, 95, 95, 95, 95, 97, 97, 97, 98, 98, 98, 99
   MyReservationsView.js0%0%0%30, 31, 35, 38, 50, 51, 53, 58, 67, 67, 67, 68, 70
   Navigation.js93.33%83.33%91.30%29, 47, 63
   PantryAdminView.js100%20%45.45%28, 36, 37, 41, 43, 47
   PantryCard.js0%0%0%16, 18, 22, 23, 37, 38, 39, 40, 41, 43, 47, 47, 47, 48, 50, 51
   PantryDetailView.js12.50%10%28.57%100, 102, 103, 108, 116, 128, 130, 132, 156, 157, 167, 168, 169, 179, 194, 206, 208, 225, 34, 42, 43, 53, 54, 55, 55, 55, 56, 56, 56, 57, 59, 68, 69, 73, 75, 90, 90, 90, 91, 94, 98, 99
   ProfileView.js19.23%13.33%12.24%100, 103, 103, 103, 104, 107, 110, 118, 119, 137, 14, 155, 16, 17, 171, 18, 187, 19, 20, 203, 21, 219, 22, 23, 235, 251, 261, 275, 292, 293, 31, 33, 46, 46, 46, 47, 47, 47, 47, 47, 47, 47, 47, 47, 47, 59, 60, 61, 62, 65, 67, 67, 67, 68, 71, 71, 71, 72, 75, 75, 75, 76, 79, 79, 79, 80, 83, 83, 83, 84, 87, 87, 87, 88, 91, 91, 91, 92, 95, 95, 95, 96, 99, 99, 99
src/components/Authentication
   LoginView.js87.50%100%100%31
   SignupView.js0%0%0%100, 103, 103, 103, 104, 107, 110, 118, 131, 133, 133, 133, 134, 137, 137, 137, 139, 142, 142, 142, 144, 147, 147, 147, 149, 152, 152, 152, 154, 157, 157, 157, 158, 161, 161, 161, 162, 165, 165, 165, 166, 169, 169, 169, 170, 173, 173, 173, 174, 177, 177, 177, 178, 181, 181, 181, 183, 186, 187, 187, 187, 189, 197, 197, 197, 198, 201, 201, 201, 202, 216, 234, 24, 250, 268, 284, 29, 300, 31, 316, 332, 342, 356, 372, 387, 405, 421, 437, 44, 453, 46, 46, 46, 469, 47, 47, 47, 47, 47, 47, 47, 47, 47, 47, 485, 501, 511, 525, 541, 542, 59, 60, 61, 62, 65, 67, 67, 67, 68, 71, 71, 71, 72, 75, 75, 75, 76, 79, 79, 79, 8, 80, 83, 83, 83, 84, 87, 87, 87, 88, 9, 91, 91, 91, 92, 95, 95, 95, 96, 99, 99, 99
src/components/modals
   AddItemModal.js100%100%100%
   OneClickReserveModal.js100%40%57.14%26, 29, 34
   ViewRsvnMsgModal.js50%50%75%40, 43

@klmui
Copy link
Member

klmui commented Apr 30, 2021

Needs to merged main again. Test if everything works @JSutanto19. Make sure all of the tests pass.

@klmui
Copy link
Member

klmui commented Apr 30, 2021

@JSutanto19 Can you merge with main again and remove the merge conflicts? Then, manually test signing up and editing your profile to see if they work.

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.

2 participants