Update subject line of storage request to include email address

[bjester]

Background

Studio users can request additional storage for their account through their account settings. This process sends an email to a static address of a Google group with the information.

Desired behavior

The subject line should say Kolibri Studio storage request from <user account's email address>. Since this is a small change, adding some test coverage on the code would be worthwhile.

Current behavior

The subject line always says Kolibri Studio storage request. There is no test coverage.

Value add

it would make it easier to track responses in the Google Group

From @laurenlichtman

[bjester]

The code in question:

studio/contentcuration/contentcuration/views/settings.py

Line 186 in aa92a57

"Kolibri Studio storage request",

[sanketkanade]

can we Update the code to dynamically generate the subject line with the user account's email address
doing this modification : subject = f"Kolibri Studio storage request from {user_email}"
then we can just write test cases to ensure that the email subject line is being generated correctly.
def test_storage_request_email_subject():
# Mock user email
user_email = "user@example.com"

# Call the function or method responsible for generating the email subject
subject = generate_storage_request_subject(user_email)

# Assert that the subject matches the expected format
assert subject == f"Kolibri Studio storage request from {user_email}"

[sanketkanade]

is this approach right

[bjester]

Hi @sanketkanade, since this is such a small change, and testing that a string is formatted as expected is so trivial, it's preferred to go a bit further and add test coverage on StorageSettingsView which would include the change. I think with some mocking, it should be a small amount of additional effort.

[sanketkanade]

Yeah sure may I work on this task

[MisRob]

Sound so :)! Thanks @sanketkanade, I will assign you.

[Abhishekzod007]

Hi @sanketkanade are you still working on this
?

[Abhishekzod007]

Hi @bjester, can I take up this issue now
I reckon Sanket is not planning on continuing this