Closed
Bug 445709
Opened 17 years ago
Closed 17 years ago
A patch which corrects some affiliate links. Needs reviewing.
Categories
(Websites Graveyard :: spreadfirefox.com, defect, P1)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: paul, Assigned: abuchanan)
References
()
Details
Attachments
(1 file)
|
2.79 KB,
patch
|
cpollett
:
review+
|
Details | Diff | Splinter Review |
Please review this patch which corrects a number of affiliate links
| Reporter | ||
Updated•17 years ago
|
Summary: Patch which corrects a number of affiliate links needs reviewing → A patch which corrects some affiliate links. Needs reviewing.
Updated•17 years ago
|
Attachment #329985 -
Flags: review?(cpollett)
Comment 1•17 years ago
|
||
Comment on attachment 329985 [details] [diff] [review]
sfx_affiliates.module patch
Hmm. I am getting the feeling I am reviewing patches which do almost the same thing. As with 447596 please change <br> to <br /> otherwise looks good.
Attachment #329985 -
Flags: review?(cpollett) → review+
| Reporter | ||
Comment 2•17 years ago
|
||
Sending sfx_affiliates/sfx_affiliates.module
Transmitting file data .
Committed revision 17370.
I can't see those changes on stage that should be present immediately without clearing the drupal cache (it's been over 15 minutes)
Would you please confirm that SVN is being checked out to stage @ https://spreadfirefox.authstage.mozilla.com
Thanks, Paul
Comment 3•17 years ago
|
||
Paul - did your updates appear?
| Reporter | ||
Comment 4•17 years ago
|
||
@Alex
I can't see the change ..
+ 'title' => t('Welcome to the Affiliate Program!')
The other changes i will not be able to see until the cron page is run to regenerate the affiliate blocks with fresh content.
Comment 5•17 years ago
|
||
Paul/Juliana: shouldn't it say Affiliates instead of Affiliate?
(In reply to comment #5)
> Paul/Juliana: shouldn't it say Affiliates instead of Affiliate?
>
Yes, we're trying to use "Affiliates" consistently throughout the program.
Updated•17 years ago
|
Priority: -- → P1
| Reporter | ||
Comment 7•17 years ago
|
||
Can we please push the latest revision of the sfx_affiliates.module to production..
http://viewvc.svn.mozilla.org/vc/projects/spreadfirefox.com/trunk/modules/sfx/sfx_affiliates/sfx_affiliates.module?revision=17555&view=markup
| Assignee | ||
Updated•17 years ago
|
Assignee: nobody → buchanae
| Assignee | ||
Comment 8•17 years ago
|
||
patch was committed to trunk in r17555 by paul
http://viewvc.svn.mozilla.org/vc?view=rev&revision=17555
| Assignee | ||
Updated•17 years ago
|
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Comment 9•17 years ago
|
||
Verified FIXED on https://spreadfirefox.authstage.mozilla.com/affiliates/homepage
Status: RESOLVED → VERIFIED
Updated•14 years ago
|
Product: Websites → Websites Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•