search

GeriLife / wellbeing-client

Client-only code for Wellbeing project.
Apache License 2.0
0 stars 0 forks source link

Adding activities page #3

Closed shailee-m closed 4 years ago

shailee-m commented 4 years ago
shailee-m commented 4 years ago

Requires #2 to be completed to complete this

brylie commented 4 years ago

This looks good overall. Perhaps it would be helpful to have some instructions on how this project works with the backend, i.e. how to get set up. Also, let's get a basic login form ready soon, even if it is hard-coded to use localhost/port for Meteor instance.

shailee-m commented 4 years ago

This looks good overall. Perhaps it would be helpful to have some instructions on how this project works with the backend, i.e. how to get set up. Also, let's get a basic login form ready soon, even if it is hard-coded to use localhost/port for Meteor instance.

Will add the instructions in this PR.

shailee-m commented 4 years ago

This looks good overall. Perhaps it would be helpful to have some instructions on how this project works with the backend, i.e. how to get set up. Also, let's get a basic login form ready soon, even if it is hard-coded to use localhost/port for Meteor instance.

Done

brylie commented 4 years ago

@shailee-m is this ready for review again or are you still working on anything in this branch?

shailee-m commented 4 years ago

@shailee-m is this ready for review again or are you still working on anything in this branch?

It is ready for review

brylie commented 4 years ago

@shailee-m I am getting the following error when loading the front page:

Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at http://localhost:3000/users/login?_t=1600767557679. (Reason: header ‘access-control-allow-headers’ is not allowed according to header ‘Access-Control-Allow-Headers’ from CORS preflight response).

I can fix the issue temporarily by editing startup.js in the wellbeing app and changing the Access-Control-Allow-Headers value to *

'Access-Control-Allow-Headers': '*',
brylie commented 4 years ago

Disregard my previous message. My branch was out of sync.