Skip to content

Conversation

oscarpolanco
Copy link
Contributor

Add a folder for the issues that have been answered

Related to the issue #120

Steps for testing the site locally:

  • Go to the website folder
  • Install all dependencies using npm install
  • Start your local server using npm run start
  • Test locally at localhost

@coveralls
Copy link

coveralls commented May 22, 2019

Coverage Status

Coverage remained the same at 99.346% when pulling 1628260 on issues_awnsers into 1893006 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 99.346% when pulling c5c006c on issues_awnsers into 1893006 on master.


As a final example on the `model.js`, we can target directly to the database on the server but on the client, we need a different approach because we don't have the same ability as the server so we perform a request to obtain our data.

The error that originates this issue tell us that we have a reference to a `services/server` but doesn't have the `services/client` reference when compile.
Copy link

Choose a reason for hiding this comment

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

Suggested change
The error that originates this issue tell us that we have a reference to a `services/server` but doesn't have the `services/client` reference when compile.
The error that originates this issue tell us that we have a reference to a `services/server` but doesn't have the `services/client` reference when compiled.

or if you want it in present tense

Suggested change
The error that originates this issue tell us that we have a reference to a `services/server` but doesn't have the `services/client` reference when compile.
The error that originates this issue tell us that we have a reference to a `services/server` but doesn't have the `services/client` reference when compiles.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks!!! @fxisco

Copy link

@manuelurenah manuelurenah left a comment

Choose a reason for hiding this comment

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

🏄

@oscarpolanco oscarpolanco merged commit d82e09b into master May 24, 2019
@oscarpolanco oscarpolanco deleted the issues_awnsers branch May 24, 2019 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants