-
-
Notifications
You must be signed in to change notification settings - Fork 928
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
SAK-50440 Rubrics remove sharing of public rubrics from other sites #12839
base: master
Are you sure you want to change the base?
Conversation
Leaving this as a draft, as Christina requested that these actions should be shown only to superusers. We're working on that but we'd like to have this included in 25 so creating the PR before the freeze date. |
refreshPage() { | ||
window.location.reload(); | ||
} |
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.
This method is no longer used and it could be removed
…n case they are superusers
@@ -19,6 +19,7 @@ save=Gorde | |||
done=Egina | |||
total=Guztira | |||
confirm_remove=Ziur zaude kendu egin nahi duzula? | |||
confirm_remove_public=Ziur "{}" publikoki partekatutako errubriketatik eta webgunearen jatorritik kendu nahi duzula? |
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.
@PabloSanRoman can you check this, please?
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 @bgarciaentornos!
Best with a little change:
- confirm_remove_public= Ziur kendu nahi duzula errubrika hau, "{}", publikoki partekatutako errubriketatik eta gunetik bertatik?
this.fetchUserRoles(); | ||
} | ||
|
||
async fetchUserRoles() { |
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.
Please rename this to checkSuperUser (we're not actually getting other roles)
@GetMapping("/user/roles") | ||
public ResponseEntity<Map<String, Boolean>> getUserRoles() { |
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.
Please rename this to checkSuperUser (we're not actually getting other roles)
…etUserRoles() method to checkSuperUser()
const roles = await response.json(); | ||
this.isSuperUser = roles.isSuperUser; | ||
|
||
console.log("Is SuperUser:", this.isSuperUser); |
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.
console.log("Is SuperUser:", this.isSuperUser); | |
console.debug("Is SuperUser:", this.isSuperUser); |
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.
@adrianfish Can you take a look, please?
In the rubrics section, the public rubrics that are displayed must be able to be unpublished or deleted from any site other than the original. This is important for institutions with hundreds of shared rubrics or when the original instructor no longer belongs to the institution.
The development has been sponsored by Hotelschool.