Page MenuHomePhabricator

GitHub-style punchcards
AbandonedPublic

Authored by epriestley on Aug 21 2012, 11:49 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 21, 3:22 PM
Unknown Object (File)
Sun, Apr 21, 3:15 PM
Unknown Object (File)
Sun, Apr 21, 3:14 PM
Unknown Object (File)
Fri, Mar 29, 4:34 AM
Unknown Object (File)
Jan 21 2024, 5:45 PM
Unknown Object (File)
Jan 19 2024, 5:39 PM
Unknown Object (File)
Jan 15 2024, 5:05 PM
Unknown Object (File)
Jan 12 2024, 6:04 AM
Subscribers
Tokens
"Love" token, awarded by qgil."Like" token, awarded by nmalcolm.

Details

Reviewers
chad
Summary

I submitted a github pull request (https://github.com/facebook/phabricator/pull/189) before I remembered that I should arc diff instead. The stuff in this diff is a bit different from the gh pull request because arc lint bosses me around.

Test Plan

Yes.

Diff Detail

Branch
punchcard
Lint
Lint Warnings
SeverityLocationCodeMessage
Warningexternals/raphael/g.raphael.dot.js:1JAVELIN5`javelinsymbols` Binary Not Built
Unit
No Test Coverage

Event Timeline

epriestley edited reviewers, added: mshang; removed: epriestley.

Haha, see GitHub. I think this is pretty cool but I want to clean it up a bit -- for example, the feed query will select stories related to you currently (e.g., someone updated a revision you are a reviewer for) not just your activity, and the calculation of hours uses server time when it should use the user's local time. Raphael also has enough weird glitchy things that I'm tempted to try d3.js. I'll see if I can do a pass on this once I get a chance.

chad requested changes to this revision.Mar 24 2017, 4:26 PM
chad removed a reviewer: mshang.
chad changed the visibility from "All Users" to "Public (No Login Required)".
This revision now requires changes to proceed.Mar 24 2017, 4:26 PM

This feature might be the key to unlocking growth, but the design needs a little polish. Gently placing back in your queue for 2018.

No current plans to pursue this upstream.