Skip to content

Commit e283689

Browse files
committed
Comment about varying URL formats.
1 parent 4134422 commit e283689

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

app/mailers/report_mailer.rb

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,10 @@ def report_content
77
@reported_content = params.fetch(:reported_content)
88
study_group = @reported_content.submission.study_group
99
exercise = @reported_content.exercise
10+
11+
# NOTE: This implementation assumes the curse URL is static and does not vary per user.
12+
# This is currently valid for all known use cases. However, in dynamic environments—such as
13+
# advanced content trees in openHPI or A/B/n testing—this assumption may break.
1014
@course_url = LtiParameter.find_by(study_group:, exercise:)&.lti_parameters&.[]('launch_presentation_return_url')
1115

1216
mail(subject: I18n.t('report_mailer.report_content.subject', content_name: @reported_content.model_name.human))

0 commit comments

Comments
 (0)