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

[LEARNER] <LMS: PROFILE> Upload photo #333

Closed
cmltaWt0 opened this issue Dec 4, 2023 · 3 comments
Closed

[LEARNER] <LMS: PROFILE> Upload photo #333

cmltaWt0 opened this issue Dec 4, 2023 · 3 comments
Milestone

Comments

@cmltaWt0
Copy link

cmltaWt0 commented Dec 4, 2023

Release

Quince

Expected behavior

The profile photo should be available after upload in the Profile MFE.

Actual behavior

The profile photo doesn't appear.
500 LMS Error:

tutor_local-lms-1  |   File "/openedx/edx-platform/./openedx/core/djangoapps/profile_images/views.py", line 159, in post
tutor_local-lms-1  |     create_profile_images(uploaded_file, profile_image_names)
tutor_local-lms-1  |   File "/openedx/edx-platform/./openedx/core/djangoapps/profile_images/images.py", line 63, in create_profile_images
tutor_local-lms-1  |     storage = get_profile_image_storage()
tutor_local-lms-1  |   File "/openedx/edx-platform/./openedx/core/djangoapps/user_api/accounts/image_helpers.py", line 30, in get_profile_image_storage
tutor_local-lms-1  |     return storage_class(**config['options'])
tutor_local-lms-1  |   File "/openedx/venv/lib/python3.8/site-packages/storages/backends/s3.py", line 280, in __init__
tutor_local-lms-1  |     super().__init__(**settings)
tutor_local-lms-1  |   File "/openedx/venv/lib/python3.8/site-packages/storages/base.py", line 15, in __init__
tutor_local-lms-1  |     raise ImproperlyConfigured(
tutor_local-lms-1  | django.core.exceptions.ImproperlyConfigured: Invalid setting 'base_url' for S3Storage

Steps to reproduce

  1. Go to Account MFE and setup a birthday to be 13+
  2. Go to Profile MFE
  3. Click Upload photo
    image

Additional information

The issue can be fixed by removing

# the following non empty property is necessary in development
"base_url": "dummyprofileimagebaseurl",

in openedx-lms-common-settings.
Need to test development local deployment.

@regisb
Copy link
Contributor

regisb commented Dec 4, 2023

This is an issue of the minio plugin, right? @FahadKhalid210 can you please have a look?

@mariajgrimaldi
Copy link
Member

label: quince testing

@regisb
Copy link
Contributor

regisb commented Dec 6, 2023

This issue was resolved by @FahadKhalid210 with a fix in the tutor-minio plugin. I can confirm that I am now able to upload profile pictures:

image

Thanks Fahad!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

3 participants