Closed
Bug 704194
Opened 14 years ago
Closed 14 years ago
I cant able to vouch some account, dont have vouch button
Categories
(Participation Infrastructure :: Phonebook, defect, P2)
Participation Infrastructure
Phonebook
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: Swarnava, Assigned: tofumatt)
References
Details
(Keywords: regression)
User Agent: Opera/9.80 (J2ME/MIDP; Opera Mini/4.2.22537/26.1153; U; en) Presto/2.8.119 Version/10.54
Steps to reproduce:
Vouching not working on some non-vouched profile
| Reporter | ||
Comment 1•14 years ago
|
||
Example url https://mozillians.org/en-US/u/3d98f9ea5a
Comment 2•14 years ago
|
||
I'm seeing this as well on the profile mentioned. Is this happening on any other non-vouched accounts?
Status: UNCONFIRMED → NEW
Ever confirmed: true
Updated•14 years ago
|
Severity: normal → major
Keywords: regression
| Reporter | ||
Comment 3•14 years ago
|
||
Updated•14 years ago
|
Component: TaskBoard → mozillians.org
Product: mozillians.org → Websites
QA Contact: taskboard → mozillians-org
Target Milestone: --- → 1.2
Updated•14 years ago
|
Priority: -- → P2
| Assignee | ||
Comment 4•14 years ago
|
||
These accounts are vouched, but we're looking at the wrong variable in the profile template after some updates. We didn't catch this because it wasn't in our test suite either, but I've added it to the tests and fixed the bug.
Waiting on an r+ but it's a simple patch. This bug'll be closed when it lands on master.
Assignee: nobody → tofumatt
Comment 5•14 years ago
|
||
Commit pushed to https://github.com/mozilla/mozillians
https://github.com/mozilla/mozillians/commit/926f80615d2d98415f8f652a184e27e5732208d8
Show vouched users as Mozillians (fix bug 704194)
Crummy regression; it's in the test suite now.
Updated•14 years ago
|
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
| Reporter | ||
Comment 6•14 years ago
|
||
Is it fixed? I still seeing same error
https://mozillians.org/en-US/u/3cf9cc8d3b, https://mozillians.org/en-US/u/c90b1a7c73
| Assignee | ||
Comment 8•14 years ago
|
||
Yes, it's fixed, but it's not on production yet. I'll update this bug with a dependant push bug when we file for our next push.
| Reporter | ||
Comment 9•14 years ago
|
||
Comment 10•14 years ago
|
||
Commit pushed to https://github.com/mozilla/mozillians
https://github.com/mozilla/mozillians/commit/79faf888335b175c2e069f2985db7e5b9827beed
Fix search results (bug 704194)
Comment 11•14 years ago
|
||
QA verified on dev. Please reopen if this no the case once we get it pushed to production (today or tomorrow).
Accounts that haven't been vouched are vouchable. With the latest patch in comment 10, recently vouched accounts no longer show up in the search results as un-vouched.
Status: RESOLVED → VERIFIED
| Reporter | ||
Comment 12•14 years ago
|
||
You can find one here
https://mozillians.org/en-US/search?q=terrence
| Assignee | ||
Comment 13•14 years ago
|
||
Swarnava: Those pages both look fine to me. If your browser has stuff cached you can try shift+refresh, but that profile shows up as a "Mozillian" in both the search and profile views.
| Assignee | ||
Comment 14•14 years ago
|
||
Swarnava: Unrelated, but the production server seems to be down. I'm filing that bug; no worries.
| Reporter | ||
Comment 15•14 years ago
|
||
Yes its fixed now thanks
Updated•14 years ago
|
Component: mozillians.org → Phonebook
Product: Websites → Community Tools
QA Contact: mozillians-org → phonebook
Target Milestone: 1.2 → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•