Add the 'required star' to the 'privacy policy' checkbox of profile edit page.

VERIFIED FIXED in 2013-07-25

Status

Participation Infrastructure
Phonebook
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: giorgos, Assigned: Julia Elman)

Tracking

other
2013-07-25

Details

(Whiteboard: [kb=1049351] )

Attachments

(3 attachments)

(Reporter)

Description

5 years ago
Created attachment 777699 [details]
Mozillians: Edit Your Profile 2013-07-18 13-36-39.png
(Reporter)

Updated

5 years ago
Whiteboard: [kb=1049351]
(Assignee)

Comment 1

5 years ago
Pull request submitted here: https://github.com/mozilla/mozillians/pull/556
(Assignee)

Updated

5 years ago
Assignee: nobody → julia

Comment 2

5 years ago
Commits pushed to master at https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/0bb4f4698a4020b8f560ad6a26587a39dd4e359d
[Fix Bug 895354] Add required star to optin field.

https://github.com/mozilla/mozillians/commit/a4acd36df83510b4b2fd441df81b71be68a391f7
Merge pull request #556 from juliaelman/895354-add-asterisk-optin

[Fix Bug 895354] Add required star to optin field.

Updated

5 years ago
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2013-07-25
(Reporter)

Comment 3

5 years ago
Not displayed correctly, see attached screenshot.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Reporter)

Comment 4

5 years ago
Created attachment 778401 [details]
Mozillians: Edit Your Profile 2013-07-19 13-42-17.png
Fixed in above commit.

Funny thing is that I actually pulled this one down and looked at it before merging it, but for some reason it didn't register in my brain that having the label below the checkbox was wrong. Anyway, no big deal. Thanks for pointing it out.
Status: REOPENED → RESOLVED
Last Resolved: 5 years ago5 years ago
Resolution: --- → FIXED
Created attachment 778575 [details]
qa - verified on stage

QA verified on stage - good catch! The privacy policy acceptance checkbox /now/ has a red '*' beside it denoting it is required.

Updated

5 years ago
Status: RESOLVED → VERIFIED
OS: Linux → All
Hardware: x86_64 → All
You need to log in before you can comment on or make changes to this bug.