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

fix: client was not receiving data from server in local dev #34

Merged
merged 2 commits into from
Jun 25, 2019

Conversation

pietrop
Copy link
Contributor

@pietrop pietrop commented Jun 25, 2019

Is your Pull Request request related to another issue in this repository ?

#33

Describe what the PR does

fixes #33 by adding this.projectsUrl inside constructor of APIWrapper class

State whether the PR is ready for review or whether it needs extra work

Ready for review

Additional context

NA

fixed by adding this.projectsUrl inside constructor of APIWrapper class
src/ApiWrapper/index.js Outdated Show resolved Hide resolved
BBC News Labs - (0) Digital Paper Edit - Sprint Board automation moved this from Review in progress to Reviewer approved Jun 25, 2019
@pietrop pietrop merged commit da0f673 into master Jun 25, 2019
BBC News Labs - (0) Digital Paper Edit - Sprint Board automation moved this from Reviewer approved to Done Jun 25, 2019
@pietrop pietrop deleted the fix-client-server-comunication branch June 25, 2019 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Client doesn't communicate to the server in local dev
2 participants