Skip to content
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

praxisdigital/moodle-block_sharing_cart#118 #186

Merged
merged 13 commits into from
Feb 5, 2024

Conversation

frederikmillingpytlick
Copy link
Collaborator

Merge praxisdigital/moodle-block_sharing_cart#118

sampraxis and others added 13 commits December 19, 2023 13:12
Fix restore settings that cause restore issue

Added moodle log when a sharing cart item got backup, restored or deleted
Improve backup/restore process by using adhoc task

Fix session lock error
Fix incorrect data type in restore controller
Limit to min. 100 characters in sharing cart item name
Add ellipsis when sharing cart item is exceed 97 characters in UTF-8 encoding
Improve sharing cart styling

Fix restore in async mode, should rerun if backup hasn't been created
Fix backup controller try to delete backup after the backup process is done
#118: Add feature: async backup/restore - testet on Moodlefourtwo, both asyncron and normal copy, with and with out drag n drop, with and without user data.
Fix upgrade script to clean up deleted user records

Add event listener on user_deleted to remove user sharing cart items without waiting for privacy provider
#118: Add feature - async backup/restore
Fix upgrade script - MYSQL with danish cannot update character that is out of danish character range
@donhinkelman donhinkelman merged commit c8c2284 into donhinkelman:master Feb 5, 2024
0 of 4 checks passed
@donhinkelman
Copy link
Owner

Thank you, Frederik. I merged this pull request. Is this a good time to make a Sharing Cart 4.3 release in the Plugins Database?

@frederikmillingpytlick
Copy link
Collaborator Author

@donhinkelman sure, just be aware that from now on, the sharing cart requires a minimum of PHP 7.4

@donhinkelman
Copy link
Owner

Thanks for pointing that out. I would have missed that important point.

@frederikmillingpytlick
Copy link
Collaborator Author

I just found a quick thing, could you wait with creating a new release? - I will let you know when it is fixed

@frederikmillingpytlick
Copy link
Collaborator Author

Everything looks good now :)

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