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

Added Mapable - Web Application #330

Merged
merged 1 commit into from
Oct 28, 2021

Conversation

atom19-i
Copy link
Contributor

Description

Mapable is map based application built using html, css and javascript along with mapbox API. One can find real time traffic details over the route. it can also find the best route between source point and destination point via cycling, walking or driving.

Fixed #329

Type of change

  • New feature (non-breaking change which adds functionality)

Checklist:

  • My code follows the style guidelines(Clean Code) of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have created a helpful and easy to understand README.md
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added screenshots(if any) that prove my fix is effective or that my feature works

@ghost
Copy link

ghost commented Oct 22, 2021

Changes to your CodeSee Architecture Map:

View a CodeSee Map of these changes

View this PR diagram as an interactive map

View more CodeSee Maps

Legend

CodeSee Map Legend

Copy link
Owner

@bislara bislara left a comment

Choose a reason for hiding this comment

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

Good work mate. Well done and sorry the delay in reviewing

So, basically Mapbox renders all the UI to enter the coordinates and show the paths and all and there is nothing more to do in client side. just need to add the div with map id. Correct me if I am wrong.

Also please add your project name,desc and author in https://github.com/bislara/code-n-stitch/blob/master/JavaScript/README.md

@atom19-i
Copy link
Contributor Author

@bislara thanks and there is bit more to this code. mapbox is not quite easy to use. it sure gives major functionality to the project but as you can see in script.js, i had to define some attributes.

@bislara bislara merged commit d3fa015 into bislara:master Oct 28, 2021
@github-actions
Copy link

Thanks very much for contributing! Your pull request has been merged 🎉 Hoping for more contribution from you in future.
Do give a ⭐ if you like this project

Happy hacking and Learning ✌🤟
Thanks again! ✨

@atom19-i
Copy link
Contributor Author

@bislara please add hacktoberfest-accepted label for this merge

@bislara
Copy link
Owner

bislara commented Oct 29, 2021

Done @atom19-i . Sorry missed it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Mapable - Web application
2 participants