-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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 additional related "subtags" upon adding new tags #9401
Conversation
Thanks for opening this pull request! This space is protected by our Code of Conduct - and we're here to help. |
Codecov Report
@@ Coverage Diff @@
## main #9401 +/- ##
=======================================
Coverage ? 79.70%
=======================================
Files ? 98
Lines ? 5958
Branches ? 0
=======================================
Hits ? 4749
Misses ? 1209
Partials ? 0 |
I'm sorry I made so many commits to change just one file...I followed the git-flow on this article, but for some reason, other files were modified during the process of solving the conflicts so I had to edit them back to the way they are in the upstream main. |
apologies, did this on the wrong issue 😅 |
What i meant to say was that you can check the exact test that failed by clicking "Details" on the right side of the test box, like shown here: Then it's a bit hard to read but you can scroll through to find the exact error:
So, something must've gone wrong with Line 48 in 99e0157
Ahhhhhhh of course... what happened here is that we overwrote the same key twice. It was my fault in the original proposed changes: subtags['pm'] = 'particulate-matter'
subtags['pm'] = 'air-quality' Thank you @cwils021 for seeing that! Let's just remove that one line - the 2nd time, where we replace Also wanted to note that i'm sorry you had some trouble with the git process, but in the end everything in https://github.com/publiclab/plots2/pull/9401/files looks good, so you got it right, @aliciapaz!!! I think if we make this one small change everything should work. Thanks for your patience and apologies for the error I introduced!!! |
Co-authored-by: Jeffrey Warren <[email protected]>
Code Climate has analyzed commit f80182d and detected 0 issues on this pull request. View more on Code Climate. |
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.
@aliciapaz LGTM 🎉
Congrats on merging your first pull request! 🙌🎉⚡️ Help others take their first stepNow that you've merged your first pull request, you're the perfect person to help someone else out with this challenging first step. 🙌 Try looking at this list of `first-timers-only` issues, and see if someone else is waiting for feedback, or even stuck! 😕 People often get stuck at the same steps, so you might be able to help someone get unstuck, or help lead them to some documentation that'd help. Reach out and be encouraging and friendly! 😄 🎉 Read about how to help support another newcomer here, or find other ways to offer mutual support here. |
Awesome, thank you @aliciapaz and thank you @Tlazypanda !!!! |
* Update _node_meta.html.erb (publiclab#8097) * Delete GCI_MENTOR_GUIDELINES.md GCI discontinued * add additional related subtags * Update _node_meta.html.erb * Update babel.config.js * Update webpacker.yml * Update babel.config.js * Update babel.config.js * Update babel.config.js * Update babel.config.js * Update babel.config.js * Update app/models/node.rb Co-authored-by: Jeffrey Warren <[email protected]> Co-authored-by: Pranav Addepalli <[email protected]> Co-authored-by: Sasha Boginsky <[email protected]> Co-authored-by: Jeffrey Warren <[email protected]>
* Update _node_meta.html.erb (publiclab#8097) * Delete GCI_MENTOR_GUIDELINES.md GCI discontinued * add additional related subtags * Update _node_meta.html.erb * Update babel.config.js * Update webpacker.yml * Update babel.config.js * Update babel.config.js * Update babel.config.js * Update babel.config.js * Update babel.config.js * Update app/models/node.rb Co-authored-by: Jeffrey Warren <[email protected]> Co-authored-by: Pranav Addepalli <[email protected]> Co-authored-by: Sasha Boginsky <[email protected]> Co-authored-by: Jeffrey Warren <[email protected]>
Fixes #9398
Added more subtags in node.rb.
Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!
rake test
@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!