504226 - project art: detect and serve custom contest upload workflow

Status: VERIFIED FIXED

(Websites Graveyard :: getpersonas.com, defect)

Description

[Suneel]

Purpose of the bug is to 1) serve a customized flow for submissions that originate from student.getpersonas.com, and 2) pre-populate certain fields within the last step of the process (submission). 

Please see attached flow. Steps 2 - 4 follow the same flow as current getpersonas.com flow. Here are notes/exceptions:

1. step 1 (landing page) work is assigned under 504222

2. step 2 (log-in) should be the exact same as getpersonas.com

3. step 3 (legal) will include unique language relevant to this competition. (see 504223)

4. step 4 (submission) will have fields "category" (and "I am submitting a persona to" if necessary) pre-populated with the name "Student Signature". This will not be a category option for users who submit through the regular getpersonas.com flow.

Comment 1

[Suneel]

Attachment: student signature flow

Comment 2

[Suneel]

morgamic, once you've had the chance to look at this in more detail, can you comment on 1) whether you need additional information, and 2) an ETC.

we are targeting to launch this around mid-August.

Comment 3

[azehren]

New target launch date is August 31.

Comment 4

[Sean Martell]

Attachment: SSC HTML, CSS + Images

All files attached.

Comment 5

[azehren]

Attachment: Legal page directons

Legal page directions for step 3.

Comment 6

[Alex Buchanan [:abuchanan]]

I've checked in the initial code for the SSC pages,

http://sm-personas01.mozilla.org/student/
(^^ needs VPN)

I'm working on updating the judges page, legal copy, and connecting with the legal and upload forms specific to this competition.

Comment 7

[Alex Buchanan [:abuchanan]]

Attachment: patch v1

the following SQL creates the needed table,

CREATE TABLE `ssc_personas` (`persona_id` int(11) not null primary key);

Comment 8

[Stephen Donner [:stephend] Not actively reading bugmail]

I can't turn this around by Tuesday since I don't have the site up, but I'll do my best when we do have /contest (awaiting review).

Comment 9

[Toby Elliott [:telliott]]

Comment on attachment 400116 [details] [diff] [review]
patch v1

looks good

Comment 10

[Alex Buchanan [:abuchanan]]

telliott, could you help me apply this patch to stage, please?

1) run the needed SQL on the stage DB
2) create a URL alias for /contest -> /upload?ssc 

(if #2 isn't easy, let me know and I'll just change the links to use upload?ssc)

Once that's done, I'll check in the patch.

Thanks

Comment 11

[Stephen Donner [:stephend] Not actively reading bugmail]

QA is still waiting on comment 10, and we're already behind -- can we get this done, please?

Comment 12

[azehren]

Assuming this will not launch today?

Comment 13

[Alex Buchanan [:abuchanan]]

oremj ran the SQL, and I checked in my patch.

I tried setting up a .htaccess file to connect /contest to /upload?ssc, but the change isn't working on stage, so far.

I'll work with telliott or IT to get it working.

(In reply to comment #12)
> Assuming this will not launch today?

It's looking that way, yes.

Comment 14

[Toby Elliott [:telliott]]

config has been rewritten. It doesn't use htaccess, but localhttpd.conf. You need to add 

RewriteRule ^/([a-zA-Z-]+/)?contest$ /$1upload?ssc [R,L]

to the install.

Comment 15

[Alex Buchanan [:abuchanan]]

(In reply to comment #14)
> config has been rewritten. It doesn't use htaccess, but localhttpd.conf. You
> need to add 
> 
> RewriteRule ^/([a-zA-Z-]+/)?contest$ /$1upload?ssc [R,L]
> 
> to the install.

Deleting       server/.htaccess
Sending        server/localhttpd-dist.conf
Transmitting file data .
Committed r51562.

Comment 16

[Alex Buchanan [:abuchanan]]

(Krupa, this explains bug 517014)

I should clarify, the workflow has changed slightly:

* The user is still presented with a custom legal page
** custom Terms of Service
** There is no option to choose a license, i.e. "How Would You Like to Share Your Personas Design?"
 
After this point, the upload workflow stays the same, in the end user's eyes.

Instead of pre-populating form fields, we've added a hidden flag that marks the upload as being a Student Signatures submission.  (This was easier and cleaner to implement)

This was decided via email and phone meetings between telliott, amy, suneel, and myself, but never clearly updated here.

Comment 17

[Alex Buchanan [:abuchanan]]

(In reply to comment #16)
> (Krupa, this explains bug 517014)

Sorry, I meant bug 517019

Comment 18

[Alex Buchanan [:abuchanan]]

I think it's safe to close this bug, the code has been committed to stage.
Of course, feel free to reopen if I'm wrong.

See bug 517042#c5 for details on how you can verify that an upload was marked as being a SSC submission.

Thanks.

Comment 19

[Vishal Kamdar [:vish_moz]]

Verified, FIXED.