Closed
Bug 997455
Opened 12 years ago
Closed 12 years ago
[dev] Unable to add/remove skills
Categories
(Participation Infrastructure :: Phonebook, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: mbrandt, Assigned: giorgos)
References
()
Details
(Keywords: regression, Whiteboard: [kb=1339928] [dev][fromAutomation])
Attachments
(1 file)
|
137.77 KB,
image/png
|
Details |
Dev only: unable to add new skills to a users profile (/user/edit/)
Steps to reproduce:
0. with a vouched account; goto https://mozillians-dev.allizom.org/user/edit/
1. add a new skill and save the changes (either by clicking 'Update Profile' or hitting the return key)
Expected:
The new skill is saved and displayed on the users profile
Actual:
The skill is not saved
| Reporter | ||
Updated•12 years ago
|
Summary: [dev] Unable to add/remove new skills → [dev] Unable to add/remove skills
| Assignee | ||
Updated•12 years ago
|
Assignee: nobody → giorgos
Status: NEW → ASSIGNED
| Assignee | ||
Comment 1•12 years ago
|
||
Thanks Matt!
Related bug 995950
| Assignee | ||
Updated•12 years ago
|
Whiteboard: [dev][fromAutomation] → [kb=1339928] [dev][fromAutomation]
Comment 2•12 years ago
|
||
Commits pushed to master at https://github.com/mozilla/mozillians
https://github.com/mozilla/mozillians/commit/dfd4fc14ac1e2bbac4697a38ab18f60af685ddca
[fix bug 997455] Remove skill JS validation.
Removing outdated client side skill validation code. We do validate
skills in python.
https://github.com/mozilla/mozillians/commit/793179ae332ae0e69fe4be17dfd7794d05cea6af
Merge pull request #905 from glogiotatidis/js-skill
[fix bug 997455] Remove skill JS validation.
Updated•12 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Version: other → next
| Reporter | ||
Comment 3•12 years ago
|
||
QA verified on dev - the addition and removal of skills work as expected.
Status: RESOLVED → VERIFIED
| Reporter | ||
Comment 4•12 years ago
|
||
Automation updated - xfail removed - https://github.com/mozilla/mozillians-tests/pull/144
Updated•12 years ago
|
Version: next → 2014-04.3
You need to log in
before you can comment on or make changes to this bug.
Description
•