-
Notifications
You must be signed in to change notification settings - Fork 14
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
N21-1319 ctl showing tool usage #2922
Conversation
itemToDelete.value = item; | ||
isDeleteDialogOpen.value = true; | ||
await fetchSchoolExternalToolUsage(item.id); | ||
if (!metadata.value) { |
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.
not the best way to do it from a user experience standpoint, because when a user makes an action/presses a button, they expect some responding action to happen. If I remember correctly a delay of 500ms between action and response can already cause disorientation. So when the loading takes a second the button press does not feel responsive anymore.
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.
Will be fixed with other dialog requirements like error handling in the next ticket
src/components/data-external-tool/SchoolExternalApi.composable.unit.ts
Outdated
Show resolved
Hide resolved
src/components/data-external-tool/SchoolExternalToolUsage.composable.unit.ts
Outdated
Show resolved
Hide resolved
src/components/data-external-tool/SchoolExternalToolUsage.composable.unit.ts
Outdated
Show resolved
Hide resolved
src/components/data-external-tool/SchoolExternalToolUsage.composable.unit.ts
Outdated
Show resolved
Hide resolved
Kudos, SonarCloud Quality Gate passed! |
Short Description
This PR handle the display of metadata in ExternalToolSection on the new admin page, when the admin wants to delete a school external tool.
Links to Ticket and related Pull-Requests
https://ticketsystem.dbildungscloud.de/browse/N21-1319
hpi-schul-cloud/schulcloud-server#4543
hpi-schul-cloud/superhero-dashboard#234
Changes
Data-security
Deployment
New Repos, NPM packages or vendor scripts
Screenshots of UI changes
Checklist before merging