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

Include author, if possible, when migrating to content bank #46

Merged
merged 4 commits into from
Nov 29, 2022

Conversation

sarjona
Copy link
Member

@sarjona sarjona commented Nov 29, 2022

A first approach was done in #37 and then @doctorlard improved it in #42
Thanks a lot, Jonathan for your contribution!! I've fixed the missing points (you can find them in several places, like the Conversation or the Files section in your PR: https://github.com/moodlehq/moodle-tool_migratehvp2h5p/pull/42/files ).

sarjona and others added 2 commits November 29, 2022 15:33
Content in the content bank will be migrated taking the user who
created the mod_hvp as author (when this information exists in logs).
@sarjona sarjona force-pushed the GH-36-author-catalyst branch 5 times, most recently from f3c5c99 to 5dff43f Compare November 29, 2022 16:32
@sarjona sarjona force-pushed the GH-36-author-catalyst branch 2 times, most recently from 03fb1a9 to d32c19d Compare November 29, 2022 16:59
@sarjona sarjona merged commit 86689dd into moodlehq:master Nov 29, 2022
@sarjona sarjona mentioned this pull request Nov 29, 2022
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.

Use the mod_hvp author when migrating into content bank too
2 participants