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

Robin natale robin #391 #429

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

robin-natale
Copy link
Contributor

Fixes #391 (<=== Add issue number here)

Please note that step 2 is missing as there were some differences in the code.

Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!

  • PR is descriptively titled 📑 and links the original issue above 🔗
  • tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR
  • code is in uniquely-named feature branch and has no merge conflicts 📁
  • screenshots/GIFs are attached 📎 in case of UI updation
  • ask @publiclab/reviewers for help, in a comment below

We're happy to help you get this ready -- don't be afraid to ask for help, and don't be discouraged if your tests fail at first!

If tests do fail, click on the red X to learn why by reading the logs.

Please be sure you've reviewed our contribution guidelines at https://publiclab.org/contributing-to-public-lab-software

Thanks!

@@ -1,4 +1,4 @@
Contributing to Leaflet Environmental Layers
Robin Contributing to Leaflet Environmental Layers
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a reason you added your name here?

@@ -0,0 +1,545 @@
require('jquery');
Copy link
Contributor

Choose a reason for hiding this comment

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

I am curious as to why it looks like all the lines were added. May I know the steps you took to make this PR? Thanks!

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah! I see that you created a new layercode.js file and added it to the root. The changes need to be added to the file in the src folder.

@@ -0,0 +1,545 @@
require('jquery');
Copy link
Contributor

Choose a reason for hiding this comment

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

Ah! I see that you created a new layercode.js file and added it to the root. The changes need to be added to the file in the src folder.

@robin-natale
Copy link
Contributor Author

Hey @crisner I was actually just trying to see if the git push request was working that’s why I initially added my name there. I think I did not realise that the file moved back to the root. I will move to the src folder as soon I get on hold of my laptop 🙏

@jywarren jywarren changed the base branch from master to main June 24, 2020 14:50
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.

Get data using bbox and populate popup data for opensense layer
2 participants