-
Notifications
You must be signed in to change notification settings - Fork 132
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
555 556 558 re-write of the AppD overview page #570
555 556 558 re-write of the AppD overview page #570
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.
Minor comments, looks good.
Co-authored-by: Hugh Troeger <troeger.hugh@gmail.com>
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.
Looks good but suggested some rephrasing. Happy to discuss.
Co-authored-by: openfin-johans <98814236+openfin-johans@users.noreply.github.com>
Merged suggested changes @openfin-johans |
Added in a sentence to resolve: |
@greyseer256 think I only need your (re)review for this then will merge into the consolidated branch (there will be further review of that in future) |
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.
Late response to the remaining issue...
Co-authored-by: openfin-johans <98814236+openfin-johans@users.noreply.github.com>
resolves #555, resolves #558, resolves #556
Re-write of the AppD overview page with the intention to better introduce the role and value of the AppD. The relationship between AppD records and other manifest formats is also clarified.