Ref T10748. This needs more extensive testing and is sure to have some rough edges, but seems to basically work so far.
Throwing this up so I can work through it more deliberately and make notes.
Differential D15842
Swap Repository Edit UI to new code epriestley on May 3 2016, 10:44 PM. Authored by Tags None Referenced Files
Subscribers None
Details Ref T10748. This needs more extensive testing and is sure to have some rough edges, but seems to basically work so far. Throwing this up so I can work through it more deliberately and make notes.
Diff Detail
Event TimelineComment Actions
TYPE_CREDENTIAL TYPE_SSH_LOGIN TYPE_SSH_KEY TYPE_SSH_KEYFILE TYPE_HTTP_LOGIN TYPE_HTTP_PASS TYPE_LOCAL_PATH TYPE_HOSTING Comment Actions I'm planning to run through this this morning in greater detail and see what else I can catch, then deploy it to secure and see how much stuff catches on fire. If it feels iffy I'll probably skip promoting this week, since these changes are broad and high-impact for almost all users and the amount of affected surface area is huge. The major weaknesses I'm aware of are: Repository Creation Walkthrough: When you create a new repository, you currently get dumped pretty unceremoniously on the manage screen with the repository inactive. There's no hinting about what to do next. What this UI should say is:
Basically, the "go through 5 pages of options" flow has been replaced with "1.5 pages > inactive repository > configure whatever else you want > activate repository". Supporting Instructions: The old interfaces had a lot of support instructions which I haven't brought over yet. Most of these aren't critical, some of them are out of date, and bringing them over gives me another chance to run through everything, so I'm planning to do this later on. Comment Actions
Comment Actions
Comment Actions
Comment Actions This is deployed on secure with only one minor hiccup (deploying it prevented the deployment process from finishing because it deployed before it applied the migration) which won't affect other installs. Nothing appears immediately broken. |