Skip to content

Conversation

@jumarub
Copy link
Contributor

@jumarub jumarub commented Apr 4, 2025

No description provided.


private String gradebookGroupEnabledCache = "org.sakaiproject.tool.gradebook.group.enabled";
private String gradebookGroupInstancesCache = "org.sakaiproject.tool.gradebook.group.instances";
private String matchingUserGradebookItemCache = "org.sakaiproject.tool.gradebook.group.user_gradebookItem";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why are you removing the cache?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because we didn't use it at all, was added in the multi gradebook feature

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that's from another commit that got carried on, but it's already incorporated

@jumarub jumarub requested a review from ottenhoff April 10, 2025 06:20
@ern ern changed the title SAK-51189: Samigo selector sync rendered fix SAK-51189 Samigo gb selector sync rendered Apr 10, 2025
@ern ern changed the title SAK-51189 Samigo gb selector sync rendered SAK-51189 Samigo gb selector render respective (un)published settings Apr 10, 2025
@ern
Copy link
Contributor

ern commented Apr 10, 2025

@jumarub can you rebase this PR with updates from master as the build is stuck on this one

@jesusmmp jesusmmp merged commit d69403e into sakaiproject:master Apr 11, 2025
5 checks passed
ern pushed a commit that referenced this pull request Apr 15, 2025
…#13549)

Co-authored-by: Sam Ottenhoff <[email protected]>
Co-authored-by: Jesús María Méndez Pérez <[email protected]>
(cherry picked from commit d69403e)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants