In revision d44bd16 we for some unknown reason removed public function checkoutInfo(TMGMTJob $job) { which contains the poll button. So reading it back.

CommentFileSizeAuthor
#6 tmgmt_mygengo-poll_tests-1969390-2.patch3.25 KBblueminds
#5 tmgmt_mygengo-poll_tests-1969390-1.patch4.63 KBblueminds
tmgmt_mygengo-readd_poll_button.patch710 bytesblueminds
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

blueminds’s picture

blueminds CreditAttribution: blueminds commented
Status: Active » Needs review
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Title: Re-add poll button » Add tests: Poll button
Issue tags: +Needs tests

Ups, commited and pushed. Sounds like we need tests ;)

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Status: Needs review » Active
miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker commented

Core related #1969434: Poll / Pull translator button

blueminds’s picture

blueminds CreditAttribution: blueminds commented
Status: Active » Needs review
FileSize
tmgmt_mygengo-poll_tests-1969390-1.patch4.63 KB

Here come the tests...

blueminds’s picture

blueminds CreditAttribution: blueminds commented
FileSize
tmgmt_mygengo-poll_tests-1969390-2.patch3.25 KB

...and once again with all tests enabled ;)

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Title: Add tests: Poll button » Re-add Poll button
Status: Needs review » Fixed

Looks, good, commited.

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