1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2025-02-06 11:58:30 +01:00
phorge-phorge/resources
epriestley a246c85c6b Use ApplicationTransactions and CustomField to implement build steps
Summary:
Ref T1049. Fixes T4602. Moves all the funky field stuff to CustomField. Uses ApplicationTransactions to apply and record edits.

This makes "artifact" fields a little less nice (but still perfectly usable). With D8599, I think they're reasonable overall. We can improve this in the future.

All other field types are better (e.g., fixes weird bugs with "bool", fixes lots of weird behavior around required fields), and this gives us access to many new field types.

Test Plan:
Made a bunch of step edits. Here's an example:

{F133694}

Note that:

  - "Required" fields work correctly.
  - the transaction record is shown at the bottom of the page.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T4602, T1049

Differential Revision: https://secure.phabricator.com/D8600
2014-03-25 16:08:40 -07:00
..
builtin Add an icon+background selector for project images 2013-10-17 09:32:34 -07:00
celerity Make Harbormaster input and output artifacts more explicit 2014-03-25 16:02:34 -07:00
chatbot Improve some documentation/examples for bot stuff 2013-02-14 12:47:39 -08:00
font Made Meme Generator 2013-01-19 18:43:43 -08:00
sprite Mail icon for email lists in typeahead 2014-02-17 10:06:16 -08:00
sql Use ApplicationTransactions and CustomField to implement build steps 2014-03-25 16:08:40 -07:00
sshd Fix two issues with shell/config scripts for hosted repositories 2013-12-04 16:45:54 -08:00