CommentFileSizeAuthor
#2 site-status-message-token-support-3025187-1.patch2.66 KBevgeny.chernyavskiy

Comments

evgeny.chernyavskiy created an issue. See original summary.

evgeny.chernyavskiy’s picture

evgeny.chernyavskiy commented
StatusFileSize
new site-status-message-token-support-3025187-1.patch2.66 KB
evgeny.chernyavskiy’s picture

evgeny.chernyavskiy commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: site-status-message-token-support-3025187-1.patch, failed testing. View results

evgeny.chernyavskiy’s picture

evgeny.chernyavskiy commented

Not sure why this patch is failing. Could maintainer(s) take a look?

drupalgideon’s picture

drupalgideon
Location London
commented
Status: Needs work » Fixed

Thanks for this. I've taken a look and it's a good suggestion.

I got the tests to run locally so I will add this (one minor change - just to say that the field supports tokens) and I've added a new test for tokens. too.

I will get this in the DEV branch shortly for review.

  • 674c723 committed on 7.x-1.x 
    Issue #3025187 by DrupalGideon...
  • evgeny.chernyavskiy authored b206fc1 on 7.x-1.x
    Issue #3025187 by evgeny.chernyavskiy: Add support for tokens in site...

Status: Fixed » Closed (fixed)

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