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
Changes to GatewayResourceProfile #42
Changes to GatewayResourceProfile #42
Conversation
…ewayResourceProfile
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.
Hi @vivekshresta , this looks pretty good. I've requested some changes, let me know if you have any questions.
@@ -38,6 +41,21 @@ export default { | |||
required: true | |||
} | |||
}, | |||
methods: { | |||
handleTypedInput() { | |||
const evt = window.event; |
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 event is passed as the first parameter to event handlers in Vue. Please use that instead of window.event
which is deprecated.
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.
@@ -177,6 +183,13 @@ export default { | |||
} | |||
return "..."; | |||
}, | |||
getUserStorageQuota(token) { |
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.
Maybe rename token
to quota
or value
?
Hi @machristie , As discussed, I'm merging all my changes to a feature branch(AIRAVATA-3315-storage-quotas). Please let me know if you want me to change anything else. |
Looks good @vivekshresta |
This is the first cut as part of EPIC: https://issues.apache.org/jira/browse/AIRAVATA-3315. By adding a new entry called UserStoragePreference, an admin user can configure storage quotas for the users in the Gateway.
0 is the default value for the UserStorageQuota entry, and the frontend, instead of displaying the value 0, displays "N/A" letting know the client that there are no storage restrictions in the Gateway.
It doesn't make any sense for the storage quota entry to have a negative number. So the client-side code has validations to make sure any typed or pasted value in the UserStorageQuota entry does not store any negative values.