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

Push file to s3 bucket #10

Merged
merged 2 commits into from
Oct 24, 2024
Merged

Push file to s3 bucket #10

merged 2 commits into from
Oct 24, 2024

Conversation

tuanngocnguyen
Copy link
Collaborator

@tuanngocnguyen tuanngocnguyen commented Oct 23, 2024

Required changes in main plugin for step form validation

https://github.com/catalyst/moodle-tool_lifecycle/pull/5/files

S3 feature is optional and requires local_aws

@tuanngocnguyen tuanngocnguyen changed the title DRAFT: Push file to s3 bucket Push file to s3 bucket Oct 23, 2024
@dmitriim
Copy link
Member

Getting following error when adding a step:
Notice: Trying to access array offset on value of type null in /var/www/monash39/admin/tool/lcbackupcoursestep/classes/lifecycle/step.php on line 380

image

@dmitriim
Copy link
Member

And then after saving following:

Notice: Trying to access array offset on value of type null in /var/www/monash39/admin/tool/lcbackupcoursestep/classes/lifecycle/step.php on line 380

Notice: Undefined index: usesdkcreds in /var/www/monash39/admin/tool/lcbackupcoursestep/classes/lifecycle/step.php on line 346
Skip to main content

classes/s3/helper.php Outdated Show resolved Hide resolved
classes/s3/helper.php Outdated Show resolved Hide resolved
classes/s3/helper.php Outdated Show resolved Hide resolved
classes/s3/helper.php Outdated Show resolved Hide resolved
classes/s3/helper.php Outdated Show resolved Hide resolved
@dmitriim dmitriim merged commit e0d97fe into MOODLE_39_STABLE Oct 24, 2024
12 checks passed
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.

2 participants