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

Update to docs page code-recipes.md #345

Closed
wants to merge 2 commits into from
Closed

Conversation

franzqat
Copy link

@franzqat franzqat commented Oct 1, 2020

I don't understand the issue here #336 but I fixed some examples in code-recipes.md so they can be run correctly without errrors.

There were some calls to owm class instead of weather_manager.
Also IDs of cities have been changed to existing ones so the examples can be run correctly

Fixed examples in code-recipes.md

There were some calls to owm class instead of weather_manager.
Also Ids of cities have been changed to existing ones so the examples can be run correctly
@csparpa
Copy link
Owner

csparpa commented Oct 1, 2020

Sweet, thanks!

csparpa added a commit that referenced this pull request Oct 2, 2020
@csparpa
Copy link
Owner

csparpa commented Oct 2, 2020

@franzqat as you've issued your PR against the master branch, I've replicated the changes on the develop branch instead
Here is the commit: daaf410

Releasing those soon

thanks

@csparpa csparpa closed this Oct 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants