Add funnelcake variables on download pages

RESOLVED FIXED

Status

RESOLVED FIXED
9 years ago
3 months ago

People

(Reporter: rebron, Assigned: rebron)

Tracking

Details

Attachments

(3 attachments)

(Assignee)

Description

9 years ago
Need to copy funnel cake variables from mozilla.com.

for: 
en-US/
en-US/thunderbird/
en-US/thunderbird/all.html
/includes/config.inc.php
(Assignee)

Updated

9 years ago
Assignee: nobody → rebron
(Assignee)

Comment 1

8 years ago
Created attachment 440236 [details]
patch to web site for fc variable
Attachment #440236 - Attachment mime type: application/octet-stream → text/plain
(Assignee)

Updated

8 years ago
Attachment #440236 - Flags: review?(bugzilla)
Comment on attachment 440236 [details]
patch to web site for fc variable

>Index: trunk/includes/config.inc.php
>===================================================================
>--- trunk/includes/config.inc.php	(revision 66192)
>+++ trunk/includes/config.inc.php	(working copy)
>@@ -88,3 +88,16 @@
> $config['gmap_api_key'] = 'ABQIAAAAJxF9iCN32uVbCYVf9GYBZRR4RvuhIVMVAVGkTLProETI91L3NxS9Bzn4k4gCwBbc2ls3ZXu0Qdt5yg';
> 
> ?>
>+
>+/**
>+ * Funnelcake suffix.  If a Funnelcake run is happening set this variable to the
>+ * appropriate suffix, otherwise leave blank.  This only affects:
>+ *      /en-US/
>+ *      /en-US/thunderbird/
>+ *      /en-US/thunderbird/all.html
>+ *
>+ * Example:
>+ *  -01
>+ */
>+$config['funnelcake_suffix'] = '-01';
>+

This looks like it should be within the ?> section.

>+<?php
>+    // Funnelcake Config
>+    $_product_details_options = array();
>+    if (array_key_exists('funnelcake_suffix', $config) && !empty($config['funnelcake_suffix'])) {
>+        $_product_details_options['tweaks'] = array( '/firefox-'.LATEST_THUNDERBIRD_VERSION.'([^"]*)en-US/' => 'thunderbird-'.LATEST_THUNDERBIRD_VERSION.$config['funnelcake_suffix'].'$1en-US');

There's no firefox on our website ;-)

In general the changes look fine otherwise, but I'm assuming the rest of the functionality required is already present in the site.
Attachment #440236 - Flags: review?(bugzilla) → review-
(Assignee)

Comment 3

8 years ago
Created attachment 440311 [details]
v2

updated w/ fixes.
(Assignee)

Updated

8 years ago
Attachment #440311 - Flags: review?(bugzilla)
(Assignee)

Comment 4

8 years ago
Fixed on trunk.
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Attachment #440311 - Flags: review?(bugzilla) → review+
(Assignee)

Comment 5

8 years ago
Created attachment 440637 [details]
fc patch addition

needed to add this as well, was already r= and checked-in to trunk by gozer.
Product: Websites → www.mozilla.org

Updated

3 months ago
Product: www.mozilla.org → www.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.