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

Feature/coordinates #129

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Feature/coordinates #129

wants to merge 6 commits into from

Conversation

tashiad
Copy link
Collaborator

@tashiad tashiad commented May 6, 2021

What Type of PR Is This?

  • New Feature
  • Bug Fix
  • Testing
  • Refactoring/Clean Up

Describe your code:

What does this branch do?

  • Adds coordinates inputs, avalanche forecast iframe, and directions button to the UI in Plan and Ride
  • Adds a 5th step to the Plan section of the tour form
  • Adds API key from openweathermap.org to .env file & .gitignore
  • Creates fetch call from Open weather map one call API
  • Adds pseudocode for next steps to be taken in the Plan and Ride files

Have you tested your branch? How?

Chrome/dev tools, Postman

Do all existing unit and integration tests pass?

N/A

Does this branch require special installation steps?

Others may need to get their own API key to test? Not sure how the .env file works. Ideally back end will take care of this once they have the bandwidth.

Related Issues:

#66 #71

Screenshots:

@tashiad tashiad added the FE label May 6, 2021
@tashiad tashiad requested review from deadbelly and rachelbuchta May 6, 2021 21:55
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.

User Story: Tour Directions User Story: Tour Weather Forecast
1 participant