Migrate Maniphest Projects (i.e. the ManiphestTaskProject class) to use edge infrastructure.
Revisions and Commits
|· · ·|
|Resolved||epriestley||T5456 Built-in Feed Query "Projects" does not display all relevant objects|
|Resolved||epriestley||T5476 Move workboard relationships to a dedicated (non-edge) table|
|Resolved||epriestley||T5595 Allow all applications to filter results by project|
|Resolved||epriestley||T2628 Add project tags to Pholio, Ponder, Differential, and other applications|
|Resolved||btrahan||T5604 Properly integrate Maniphest with Subscribers|
|Resolved||None||T4973 Add project names to a header in outbound Maniphest emails|
|Open||None||T6030 Allow CustomFields to specify additional options (like "required") configurable in `<application>.fields`|
|Wontfix||joshuaspence||T5011 Convert builtin fields in Maniphest to CustomField|
|Resolved||btrahan||T5245 Migrate Maniphest Projects to use edge infrastructure|
|· · ·|
D9340 slightly related.
This is probably pretty involved, I'll take a stab at it at some point. I don't think it gives us a ton of stuff, but would let us clean up a few things.
This is mostly complete, but:
- Maniphest should implement PhabricatorProjectInterface, but picks up two "Projects" fields in the UI if it does. Its projects field currently shows board columns, so we can't just delete that copy. After T4807, we can generalize this and implement PhabricatorProjectInterface.
- It would be nice to migrate old TYPE_PROJECTS transactions to TYPE_EDGE transactions to remove leftover rendering code.
Support Impact While this is mostly resolved, it created inconsistencies and blocked other tasks.
Stealing this from you to do these bits...? Feel free to take back if you have something on it as I won't be getting started on this for a few hours at least.
Stealing this from you to do these bits...?
Go for it. First one is definitely worth doing but the the second one may be more trouble than it's worth (we only get to remove a fairly small amount of dead code, I think, and it could be a long trek to get there).
Please check https://secure.phabricator.com/maniphest/report/project/
This is the result of this change:
rP83db5965ab6c: Maniphest - introduce needProjectPHIDs