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

Change English choose_starting_point text for consistency #1992

Conversation

jenPlusPlus
Copy link

@jenPlusPlus jenPlusPlus commented Dec 6, 2017

What? Why?

We changed all text (in English) that read, "Choose your starting point" to "Choose your package"

Closes #1926

This change was needed to increase consistency. Feedback from users indicated that there was confusion between terms like profile, type, and package.

We ran into a challenge when handling this issue:

We were unable to locate the issue in the interface of the application. However, we were able to find it and change the text in the codebase. We aren't able to confirm the change in the user interface of the application without recreation steps.

We had difficulty installing dependencies using rbenv. We found success using RVM.

What should we test?

Testing is needed to verify that the user interface of the application is displaying the desired text.

jenPlusPlus and others added 2 commits December 6, 2017 14:52
My environment set up removed some necessary code from Gemfile.lock. This commit adds them back.
Copy link
Contributor

@sauloperez sauloperez left a comment

Choose a reason for hiding this comment

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

Seems good to me. Can some native English speaker confirm?

Gemfile.lock Outdated

BUNDLED WITH
BUNDLED WITH
Copy link
Contributor

Choose a reason for hiding this comment

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

can we please leave this changes out of the PR?

@daniellemoorhead
Copy link
Contributor

@oeoeaio or @mkllnk can you please stage this for @sstead to check?

@daniellemoorhead
Copy link
Contributor

Oh, also, @sauloperez will this need to be covered in transifex also?

@sauloperez
Copy link
Contributor

sauloperez commented Dec 7, 2017

Yep, we should change this copy in Transifex cc @mkllnk

Copy link
Member

@mkllnk mkllnk left a comment

Choose a reason for hiding this comment

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

Great. Thank you for your contribution. Are you able to amend some of the changes? If not, just tell us and we will take over from here.

@@ -802,9 +802,8 @@ DEPENDENCIES
whenever
wicked_pdf
wkhtmltopdf-binary

RUBY VERSION
RUBY VERSION
Copy link
Member

Choose a reason for hiding this comment

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

You probably didn't mean to change this file. Can you revert your change?

@@ -529,7 +529,7 @@ en-GB:
welcome_title: Welcome to the Open Food Network!
welcome_text: You have successfully created a
next_step: Next step
choose_starting_point: 'Choose your starting point:'
choose_starting_point: 'Choose your package:'
Copy link
Member

Choose a reason for hiding this comment

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

One thing you couldn't know, is that we change only en.yml manually. That is our source locale that gets translated with Transifex. The translations from Transifex get then merged into our code automatically. So if you could revert you changes on all other locales, that would be great. Otherwise we will face merge conflicts.

@myriamboure
Copy link
Contributor

@jenPlusPlus just wondering if you would like to finish that one? It's been open for a long time. Else let us know and we'll ask if someone can take over :-)

@myriamboure
Copy link
Contributor

Does anyone want to finish that one? Else let's just close and someone will do it at some point. Funny to see that this PR was a copy of a PR done on a fork of OFN referenced in #1926 but they didn't take transifex into account hahaha ;-)

@sauloperez
Copy link
Contributor

Let's close it for now

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.

None yet

6 participants