Follow-up from #1416372: Translator Plugin: Local User

Create README.txt

CommentFileSizeAuthor
#10 create_readme_txt_1877022_7.patch1.25 KBcgalli
#7 tmgmt-missing-pending-translations-1900926-7.patch2.67 KBcgalli
#3 create_readme_txt_1877022_3.patch2.63 KBcgalli

Comments

cgalli’s picture

cgalli commented
Issue summary: View changes

Change the reference

cgalli’s picture

cgalli commented
Issue summary: View changes

content added

miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented
Issue summary: View changes

fix link

miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented
Issue summary: View changes

Updated issue summary.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented

deleted.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented
Issue summary: View changes

Updated issue summary.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented
Title: Follow-up from #1416372: Translator Plugin: Local User » Create README.txt

Fix title.

cgalli’s picture

cgalli commented
StatusFileSize
new create_readme_txt_1877022_3.patch2.63 KB

First Draft

cgalli’s picture

cgalli commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, create_readme_txt_1877022_3.patch, failed testing.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs work » Fixed

Thanks.

The patch had windows file endings, that's why the testbot didn't like it. Make sure you use an editor where you can enforce unix line endings.

Converted and commited. We can always improve this further.

cgalli’s picture

cgalli commented
Status: Fixed » Needs review
StatusFileSize
new tmgmt-missing-pending-translations-1900926-7.patch2.67 KB

Adapted text to patch.

Linux endings should be correct.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

Wrong patch?

Status: Needs review » Needs work

The last submitted patch, tmgmt-missing-pending-translations-1900926-7.patch, failed testing.

cgalli’s picture

cgalli commented
Status: Needs work » Needs review
StatusFileSize
new create_readme_txt_1877022_7.patch1.25 KB

Wrong patch indeed. But at least the No.7 was correct ;-)

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Fixed

Committed. This should have been part of the other patch, not a re-opening of this one :)

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

Updated issue summary.