Closed
Bug 1083925
Opened 11 years ago
Closed 10 years ago
Ship revised Mentor page to teach.webmaker.org
Categories
(Webmaker Graveyard :: webmaker.org, defect)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 1104256
People
(Reporter: mattheww, Assigned: akirose, Mentored)
References
()
Details
(Whiteboard: [hold][teachmigration][goodfirstbug])
Attachments
(4 files, 3 obsolete files)
Landing page for mentors has been revised, content reduced for clarity and usability. Let's build it.
Mockup
https://redpen.io/fc4dfcfc3476a74717
Copy & Links
https://webmaker.etherpad.mozilla.org/mentor-landing-page-copy
Comment 1•11 years ago
|
||
Wex, think anyone will have cycles Friday or Monday to help implement this? We've kept the changes super simple, text and images only. Should not take more than an hour.
Flags: needinfo?(simon)
Comment 2•11 years ago
|
||
Everyone is against the wall. If Gavin or Aki have time, it'll be them. Otherwise I'm sure I can do it during my travel-to-mozfest time. I'm leaving the needinfo hanging in the meantime.
Comment 3•11 years ago
|
||
Added [goodfirstbug] in case any contributors can take it. :)
Mentor: mattheww
Whiteboard: [mozfest][oct17] → [mozfest][oct17][goodfirstbug]
| Reporter | ||
Comment 4•11 years ago
|
||
Ready to build. As Cassie said, it doesn't require any new layout or engineering.
Comment 5•11 years ago
|
||
* Simon will see whether there's someone who might be able to take this one
| Reporter | ||
Comment 6•11 years ago
|
||
This photo will work, but it's not ideal. Cassie might provide something better.
Attachment #8507080 -
Flags: feedback?(cassie)
Comment 7•11 years ago
|
||
* the [oct17] train has now left the station
* so please update with [oct31], [nov14], [nov28], etc.
| Reporter | ||
Comment 8•11 years ago
|
||
Attachment #8507909 -
Flags: review?(cassie)
Comment 9•11 years ago
|
||
* Thanks Matthew! Simon: can you assign?
Comment 10•11 years ago
|
||
Comment on attachment 8507909 [details]
new photo, more pixels. mentoring-outside.jpg
Little too tall and dark. Will upload a new version in a sec.
Attachment #8507909 -
Flags: review?(cassie) → review-
Comment 11•11 years ago
|
||
New asset here, slightly tweaked.
Attachment #8507080 -
Attachment is obsolete: true
Attachment #8507909 -
Attachment is obsolete: true
Attachment #8507080 -
Flags: feedback?(cassie)
Attachment #8507957 -
Flags: review?(mattheww)
Comment 12•11 years ago
|
||
Asset attached, also here: https://www.dropbox.com/sh/jdcaxhdbhc5g6hd/AADtKmAcbDYuLb-NcLrLqfy0a?dl=0
Comment 13•11 years ago
|
||
Summarizing the assets for this page:
Copy and links: https://webmaker.etherpad.mozilla.org/mentor-landing-page-copy
Design: https://redpen.io/fc4dfcfc3476a74717
Flag me if any info is conflicting!
Updated•11 years ago
|
Assignee: nobody → simon
Comment 15•11 years ago
|
||
* [oct17] has passed. Please update the whiteboard.
| Reporter | ||
Comment 16•10 years ago
|
||
Comment on attachment 8507957 [details]
https://redpen.io/fc4dfcfc3476a74717
photo crop looks good to me
Attachment #8507957 -
Flags: review?(mattheww) → review+
Comment 17•10 years ago
|
||
* Simon, can we get this done this sprint?
Flags: needinfo?(simon)
Whiteboard: [mozfest][oct17][goodfirstbug] → [mozfest][nov14][goodfirstbug]
Comment 18•10 years ago
|
||
* [nov14] is past -- please update to [nov28] train or later
Updated•10 years ago
|
Assignee: simon → scott
Flags: needinfo?(simon)
| Reporter | ||
Updated•10 years ago
|
Status: NEW → ASSIGNED
Comment 19•10 years ago
|
||
I would like to take this, if someone can guide me.
Comment 20•10 years ago
|
||
Attachment #8527856 -
Flags: review?(aki)
Updated•10 years ago
|
Assignee: scott → tanay1337
Mentor: mattheww → aki
Comment 21•10 years ago
|
||
* There's been a change of plans for this ticket. We would like to change the path for this page.
* Instead of going to webmaker.org/mentor, we would like it to instead go to: "teach.webmaker.org"
Comment 22•10 years ago
|
||
* Wex: can you help from here?
Flags: needinfo?(simon)
Summary: Build revised Mentor page → Ship revised Mentor page to teach.webmaker.org
Comment 23•10 years ago
|
||
Matt, created https://bugzilla.mozilla.org/show_bug.cgi?id=1104256 , assigned to Aki.
Updated•10 years ago
|
Whiteboard: [mozfest][nov14][goodfirstbug] → [mozfest][dec12][goodfirstbug]
Updated•10 years ago
|
Whiteboard: [mozfest][dec12][goodfirstbug] → [dec12][teachmigration][goodfirstbug]
| Assignee | ||
Comment 24•10 years ago
|
||
Comment on attachment 8527856 [details] [review]
https://github.com/mozilla/webmaker.org/pull/1016
Removing the R? as this is a work in progress
Attachment #8527856 -
Flags: review?(aki)
Comment 25•10 years ago
|
||
Aki, it looks like we merged the PR. Is it in staging, can Michelle review?
Flags: needinfo?(simon) → needinfo?(aki)
| Assignee | ||
Comment 26•10 years ago
|
||
There's another branch at https://github.com/gesa/webmaker.org/compare/mozilla:master...gesa:1083925/mentor-update that isn't a PR since it's not going to get merged into webmaker.org. I still don't know the details of teach.wm.o to I haven't made a PR there.
Flags: needinfo?(aki)
| Reporter | ||
Comment 27•10 years ago
|
||
If it's ready and easy, might be useful to merge with wm.org for reference in the planning of teach.wm.org.
| Assignee | ||
Comment 28•10 years ago
|
||
Assignee: tanay1337 → aki
Attachment #8527856 -
Attachment is obsolete: true
Attachment #8533274 -
Flags: review?(gavin)
Comment 29•10 years ago
|
||
Comment on attachment 8533274 [details] [review]
https://github.com/mozilla/webmaker.org/pull/1022
LGTM!
Attachment #8533274 -
Flags: review?(gavin) → review+
Comment 30•10 years ago
|
||
Commits pushed to master at https://github.com/mozilla/webmaker.org
https://github.com/mozilla/webmaker.org/commit/374badc5ad55477375378e7c518796025547c723
Fixed Bug 1083925
Fixed Bug 1083925
https://github.com/mozilla/webmaker.org/commit/b994c6c5bcd725cb4c871b34d774575118dbd145
Bug 1083925 - Ship revised Mentor page to teach.webmaker.org
https://github.com/mozilla/webmaker.org/commit/db1ef35f94ada409936868c509af4927eaab230b
Merge pull request #1022 from gesa/1083925/mentor-update
Bug 1083925 - Ship revised Mentor page
Updated•10 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 31•10 years ago
|
||
Looks good. We can remove the "meet some amazing mentors" headline, and "Community" should have a lower-case c. Thanks Aki!
| Assignee | ||
Updated•10 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
| Assignee | ||
Comment 32•10 years ago
|
||
Attachment #8535199 -
Flags: review?(jon)
Comment 33•10 years ago
|
||
Comment on attachment 8535199 [details] [review]
https://github.com/mozilla/webmaker.org/pull/1032
conditional r+ on Travis passing
Attachment #8535199 -
Flags: review?(jon) → review+
Comment 34•10 years ago
|
||
Not sure which bug to place these adjustments in, but dropdown menu and copy needs updates. From an email thread:
Pulldowns:
—"Resources" links to https://webmaker.org/en-US/resources but perhaps we can lose the current subnav on that page
—"Badges" links to http://blog.webmaker.org/badges (unless bug 1010306 is fixed)
—"Hive" links to http://hivelearningnetworks.org
Lainie would like to add to the main splash page something like “Become a catalyst in your local community. Learn more about the Hive Learning Networks model and how cities around the globe are championing digital skills and web literacy through connected learning.” [Link to http://hivelearningnetworks.org]
| Reporter | ||
Comment 35•10 years ago
|
||
Laura -
You can post navigation changes to https://bugzilla.mozilla.org/show_bug.cgi?id=1108811
The design and copy of the landing page was shipped weeks ago. The build shipped yesterday. I would file a new bug to add content about Hive.
Ping Bobby with questions. He's on point for both of these projects.
Comment 36•10 years ago
|
||
Yeah. +1. I spoke with the team and Lainie suggested those nav changes. Let's lift that request out of here though. I'll do.
| Assignee | ||
Updated•10 years ago
|
Status: REOPENED → NEW
Updated•10 years ago
|
Whiteboard: [dec12][teachmigration][goodfirstbug] → [hold][teachmigration][goodfirstbug]
Comment 37•10 years ago
|
||
I think this bug can remain closed in favour of 1104256. Is there more work that needs to be done here other than moving to teach.w.o eventually?
Flags: needinfo?(aki)
| Assignee | ||
Comment 38•10 years ago
|
||
lgtm.
Status: NEW → RESOLVED
Closed: 10 years ago → 10 years ago
Flags: needinfo?(aki)
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•