Skip to content

Conversation

@nedbat
Copy link
Contributor

@nedbat nedbat commented Jan 3, 2014

No description provided.

@nedbat
Copy link
Contributor Author

nedbat commented Jan 3, 2014

@dianakhuang @ormsbee @sarina See what I did there?

Copy link
Contributor

Choose a reason for hiding this comment

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

Not that the choice of language here is part of your PR but this sentence strikes me as terribly odd. "The decision they returned"?? They referring to the payment processor? And "decision"? Seems like a vague indirect object ("they") and a very, very weird choice of word ("decision"). Not sure if I can even parse to suggest something better.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, I changed the obvious mistake, and decided not to wade into the more treacherous waters.

@sarina
Copy link
Contributor

sarina commented Jan 6, 2014

👍

@dianakhuang
Copy link
Contributor

🚢

nedbat added a commit that referenced this pull request Jan 7, 2014
Miscellaneous fixes to strings found during i18n work.
@nedbat nedbat merged commit 549d923 into master Jan 7, 2014
@nedbat nedbat deleted the ned/misc-msg-fixes branch January 7, 2014 14:41
shimulch pushed a commit to open-craft/openedx-platform that referenced this pull request Jan 26, 2021
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.

4 participants