Skip to content

Additional media protection based on Course or Membership#2894

Merged
brianhogg merged 67 commits intodevfrom
additional-media-protection
Jul 15, 2025
Merged

Additional media protection based on Course or Membership#2894
brianhogg merged 67 commits intodevfrom
additional-media-protection

Conversation

@brianhogg
Copy link
Copy Markdown
Contributor

@brianhogg brianhogg commented Mar 14, 2025

Description

Ability to protect a file in the media library to only be accessible by a logged in user who is enrolled in a Course or Membership.

Also checks to only serve files that have the authorization filter attached.

How has this been tested?

Manually

Screenshots

CleanShot 2025-07-22 at 11  16 24@2x

CleanShot 2025-03-14 at 5  36 57@2x
CleanShot 2025-03-14 at 5  37 20@2x

Checklist:

  • This PR requires and contains at least one changelog file.
  • My code has been tested.
  • My code passes all existing automated tests.
  • My code follows the LifterLMS Coding & Documentation Standards.

brianhogg added 30 commits March 4, 2025 12:12
… uploaded file (and related sizes if an image) to the lifterlms protected folder.
…rect we can just update the value of the text input manually.
…y fetch an image if the size or icon param exists. Also check that it's an attachment post before serving it.
…he file is moved into the folder it's protected, would need to move the file out of the folder first.
@brianhogg brianhogg requested a review from ideadude as a code owner March 14, 2025 16:37
@ideadude ideadude moved this to Awaiting Review in Development Mar 14, 2025
@brianhogg brianhogg added this to the Next Available Release milestone Mar 31, 2025
@ideadude ideadude modified the milestones: Next Available Release, 8.1 Mar 31, 2025
@brianhogg brianhogg merged commit d094c66 into dev Jul 15, 2025
18 of 20 checks passed
@brianhogg brianhogg deleted the additional-media-protection branch July 15, 2025 15:37
@github-project-automation github-project-automation bot moved this from Awaiting Review to Done in Development Jul 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants