-
Notifications
You must be signed in to change notification settings - Fork 0
Dev/70/carles show details #83
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: dev/70/alex_hierarchy
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead of using our personal ones, I believe this should be the reference settings file that we use when working on code improvements for everything github/git related, which I believe is going to be most of our next body of work.
|
We also need to handle 404s when a feature is looked up that does not exist. At the backend, I'm putting the logic in place to return a 404. Angular needs to handle this and display something appropriately (ATM it just continues to show the spinning wheel) |
|
I just updated this branch with the mock-up of a more suitable JSON response, and some front-end that starts to resemble what I sketched out on paper Run |
|
Forgot to mention, the focus of the discussion on this PR has shifted to getting consensus on:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Structure seems fine to me.
Presumably additional details could be added to the details list. Not sure how we'd display that additional information since it seems we are only showing one overall colour but that's one for another day I guess...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The main idea as described in the original google doc was: "Status/health is a single square with a color code. It represents an aggregated view of a number of indicators. Upon clicking/hovering one can see the detailed decomposition (e.g. build status, branch relevance, branch staleness, ready to deploy...) "
I will add some more details for illustration and completeness.
|
I think the visual side is a great start. There's a lot of white space going to be created as a result of the way the branching works, but I'm fine with that for now. |
|
I am sure we can try and find ways to make better use of screen, and reduce white space without generating clutter. There are a few blank ares... What white space are you specifically thinking about at this moment? |
|
The white space to the right of the set of branches. I think in time other information to do with the feature could go there, so it's not a biggie. |


This is not intended to be merged, but mainly for discussion. You can also use it to add some crude visuals to the API.
When trying to glue things together in a quick first initial story detail view, I realised the JSON data structure we are generating right now (nested dictionaries) is not ideally suited for iterating in Javascript and Angular-JS templates. See my inline comments in the PR.
We may want to come up with a properly defined and documented feature JSON data structure (and in particular all attachment info: repos, branches, build results...) before going much further.