-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
UI/license autoload #11778
UI/license autoload #11778
Conversation
@@ -1,22 +1,32 @@ | |||
import { equal } from '@ember/object/computed'; | |||
import Component from '@ember/component'; | |||
import Component from '@glimmer/component'; |
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.
nice work glimmerizing
|
||
<section class="box is-sideless is-marginless is-shadowless is-fullwidth"> | ||
<span class="title is-5">Details</span> | ||
|
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.
is it intentional leaving a line space here?
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.
it was just easier to parse the sections while I was working on removing code
ui/mirage/config.js
Outdated
performance_standby_count: 0, | ||
start_time: '2020-04-28T00:00:00Z', | ||
}, | ||
// persisted_autoload: { |
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.
curious why these are commented out?
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 was testing with various configurations but wanted to make sure we didn't lose the other options
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.
Great work, a couple of non-blocking comments.
Update license page for autoloaded license.
When Autoloaded
![Screen Shot 2021-06-03 at 4 37 24 PM](https://user-images.githubusercontent.com/82459713/120866188-08796e80-c555-11eb-9548-64fb62b15f61.png)
![Screen Shot 2021-06-03 at 4 36 41 PM](https://user-images.githubusercontent.com/82459713/120866178-044d5100-c555-11eb-8e39-68456d2663d6.png)
![Screen Shot 2021-06-03 at 4 36 39 PM](https://user-images.githubusercontent.com/82459713/120866168-00b9ca00-c555-11eb-8d59-38bf8b44b7b5.png)
Still shows standby count
Shows warning if license is stored