-
-
Notifications
You must be signed in to change notification settings - Fork 127
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
Add the header for naviage to map #1189
Conversation
@AQIB-NAWAB is attempting to deploy a commit to the openbeta-dev Team on Vercel. A member of the Team first needs to authorize it. |
fix #1177 |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @AQIB-NAWAB it looks like you've still got those linting errors.
Can you run yarn run lint
in your local terminal on this branch and fix those warnings? No need to create another pr, we can squash the changes when merging.
@vnugent would you mind approving the vercel deploy for this? |
@clintonlunn Oops forgot to run that command , Just resolved the linting errors |
Hi @AQIB-NAWAB i added a few comments. In addition can you pull in the latest changes. You'll see that I think it's fine to add the |
Superseded by PR #1196 |
name: Pull request
about: Create a pull request
title: add hero alert for "Find Climbings by State " at home page
labels: ''
assignees: ''
What type of PR is this?(check all applicable)
Description
Add the header for naviage to map
Related Issues
Issue # #1177
What this PR achieves
Add a button to navigate the page to search the locations by state
Screenshots, recordings