-
Notifications
You must be signed in to change notification settings - Fork 78
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
base: main
Are you sure you want to change the base?
Conversation
Step 1 completed
@@ -1,4 +1,4 @@ | |||
Contributing to Leaflet Environmental Layers | |||
Robin Contributing to Leaflet Environmental Layers |
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.
Is there a reason you added your name here?
@@ -0,0 +1,545 @@ | |||
require('jquery'); |
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.
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!
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.
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'); |
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.
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.
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 🙏 |
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!
@publiclab/reviewers
for help, in a comment belowIf 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!