-
Notifications
You must be signed in to change notification settings - Fork 48
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
#475 About tab inside top toolbar. #506
Conversation
@@ -74,6 +79,7 @@ protected void process(DBKey dbKey, HttpServletRequest req, HttpServletResponse | |||
String suiteName = req.getParameter(Helper.SUITE_PARAM); | |||
String suiteVersion = req.getParameter(Helper.VERSION_PARAM); | |||
String formatted = req.getParameter(FORMATTED_PARAM); | |||
resp.setHeader(APP_VERSION_HEADER, bundleVersionProvider.getBundleVersion()); |
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.
👍
Other metadata are send as JSON properties. Has anyone any ideas/thoughts about this being sent as header?
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.
I think it's fine because this is some kind of "meta" information. But maybe we should prefix the custom header name with X-
(X-Application-Version
), WDYT?
CC @Skejven
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.
X-
prefix is a good idea. Sending it in the header is fine.
Description
Possibility to show simple "About" informations.
Motivation and Context
Mainly it allows users to check installed version of AET application.
Additionally it shows "Contact and Commercial Support" section and footer.
Issue: #475
Screenshots (if appropriate)
Toolbar element:
Popup:
Upgrade notes (if appropriate)
Types of changes
Checklist:
I hereby agree to the terms of the AET Contributor License Agreement.