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

changed function "t" to "translation" on line 23 and 26. #6768

Closed
wants to merge 3 commits into from

Conversation

priyajitm
Copy link

@priyajitm priyajitm commented Nov 12, 2019

changed function "t" to "translation" on line 23 and 26.

Fixes #6736

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 -- or run tests locally with rake test
  • 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!

changed function "t" to "translation" on line 23 and 26.
@welcome
Copy link

welcome bot commented Nov 12, 2019

Thanks for opening this pull request! This space is protected by our Code of Conduct - and we're here to help.
Dangerbot will test out your code and reply in a bit with some pointers and requests.
Also please refer here for installation help 💿
There may be some errors, but don't worry! We'll work through them with you! 👍🎉😄
It would be great if you can tell us your Twitter handle so we can thank you properly?

@priyajitm
Copy link
Author

priyajitm commented Nov 12, 2019 via email

@nstjean
Copy link
Contributor

nstjean commented Nov 12, 2019

I'm not sure why that's failing, it's been doing that a lot lately. It should be an issue, nothing you did!

Can you change the Issue # at the top in your post to the issue that this fixes? #6736 Thanks!

Copy link
Contributor

@nstjean nstjean left a comment

Choose a reason for hiding this comment

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

This works!
FireShot Capture 061 - 🎈 Public Lab_ Stats - localhost

@priyajitm priyajitm changed the title updated plots2\app\views\grids\_thumbnail.html.erb updated plots2\app\views\grids\_thumbnail.html.erb ti fix issue #6736 Nov 12, 2019
@priyajitm priyajitm changed the title updated plots2\app\views\grids\_thumbnail.html.erb ti fix issue #6736 issue #6736 fix - updated plots2\app\views\grids\_thumbnail.html.erb Nov 12, 2019
@jywarren jywarren changed the title issue #6736 fix - updated plots2\app\views\grids\_thumbnail.html.erb changed function "t" to "translation" on line 23 and 26. Nov 12, 2019
@jywarren
Copy link
Member

Awesome! I'm working on another fix for the thumbnails file, so I'm going to hold off a little bit before merging this, but this looks great!

@plotsbot
Copy link
Collaborator

2 Warnings
⚠️ It looks like you merged from master in this pull request. Please rebase to get rid of the merge commits – you may want to rewind the master branch and rebase instead of merging in from master, which can cause problems when accepting new code!
⚠️ There was an error with Danger bot’s Junit parsing: No JUnit file was found at output.xml
3 Messages
📖 @priyajitm Thank you for your pull request! I’m here to help with some tips and recommendations. Please take a look at the list provided and help us review and accept your contribution! And don’t be discouraged if you see errors – we’re here to help.
📖 It looks like you haven’t marked all the checkboxes. Help us review and accept your suggested changes by going through the steps one by one. If it is still a ‘Work in progress’, please include ‘[WIP]’ in the title.
📖 #
Screenshots 📸 (click to expand)

6768-test_questions.png

6768-test_embeddable_grids.png

6768-test_signup.png

6768-test_viewing_the_settings_page.png

6768-test_tag_by_author_page.png

6768-test_wiki_page_with_inline_grids.png

6768-test_stats.png

6768-test_viewing_the_dashboard.png

6768-test_searching_an_item_from_the_homepage.png

6768-test_questions_shadow.png

6768-test_login_modal.png

6768-test_profile_page.png

6768-test_comments.png

6768-test_tags.png

6768-test_signup_modal.png

6768-test_wiki.png

6768-test_methods.png

6768-test_tag_page.png

6768-test_blog_page_with_location_modal.png

6768-test_tag_wildcard.png

6768-test_embeddable_thumbnail_grids.png

6768-test_front_page_with_navbar_search_autocomplete.png

6768-test_login.png

6768-test_viewing_the_dropdown_menu.png

6768-test_viewing_question_post.png

6768-test_mobile_displays.png

6768-test_simple-data-grapher_powertag.png

6768-test_front.png

6768-test_question_page.png

6768-test_tag_contributors_page.png

6768-test_blog.png

6768-test_people.png

6768-test_wiki_revisions.png

Learn about automated screenshots

Generated by 🚫 Danger

@SidharthBansal
Copy link
Member

@jywarren what is the status of related PRs?

@jywarren
Copy link
Member

jywarren commented Jan 7, 2020 via email

@nstjean
Copy link
Contributor

nstjean commented Jan 9, 2020

Hi @SidharthBansal ! The full list is here #6579 - templates that have a FTO created are checked off with links to the issues. It is up to date. Do you want me to mark off which have been merged and which haven't?

@SidharthBansal
Copy link
Member

yeah, please

@nstjean
Copy link
Contributor

nstjean commented Jan 13, 2020

@SidharthBansal Ok I'll get that done!

@SidharthBansal
Copy link
Member

Any progress on this?
Just checking... no hurry

1 similar comment
@SidharthBansal
Copy link
Member

Any progress on this?
Just checking... no hurry

@nstjean
Copy link
Contributor

nstjean commented Jan 17, 2020

Ahh, sorry! I got caught up in testing issues. I'll go through it today!

@codecov
Copy link

codecov bot commented Mar 5, 2020

Codecov Report

❗ No coverage uploaded for pull request base (master@eaf36b1). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #6768   +/-   ##
=========================================
  Coverage          ?   81.90%           
=========================================
  Files             ?       97           
  Lines             ?     5615           
  Branches          ?        0           
=========================================
  Hits              ?     4599           
  Misses            ?     1016           
  Partials          ?        0           

@cesswairimu
Copy link
Collaborator

This PR has been open for a long time and no activity/ reviews requested has not been addressed. We understand you might be busy and engaged on other things. I am closing this for now...please feel free to reopen if you get some time and would like to finish this. Rem to check if its still available before you reopen. Thanks

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.

change function t to translation of views/grids/_thumbnail
7 participants