I've happened upon a few pieces of text that I feel could be improved, mostly for better grammar. (This isn't an exhaustive review of the text.) Things I'd like to see changed:

  • "update all fields referencing to the old ones" -> "update all fields referencing the old ones"
  • "oups" -> "oops"
  • "something does not seem to go right" -> "something did not seem to go right"
  • "are you sure want to merge" -> "are you sure you want to merge"

A patch will be attached shortly.

CommentFileSizeAuthor
#4 after.png13.39 KBvolkswagenchick
#2 text_improvements-2948523-2.patch2.01 KBkpaxman

Comments

kpaxman created an issue. See original summary.

kpaxman’s picture

kpaxman commented
StatusFileSize
new text_improvements-2948523-2.patch2.01 KB
legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
commented
Status: Active » Needs review
volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new after.png13.39 KB

Looked over code and grammar looks good. Patch applied locally and the UI fixes are complete. Screenshot attached. marking RTBC

  • bucefal91 committed 5378c69 on 7.x-1.x authored by kpaxman 
    Issue #2948523 by kpaxman, volkswagenchick: Improve some module text
bucefal91’s picture

bucefal91 commented
Status: Reviewed & tested by the community » Fixed

Thank you, guys!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.