Skip to content
This repository has been archived by the owner on Sep 29, 2024. It is now read-only.

Added secure file media type and config files #123

Open
wants to merge 14 commits into
base: develop
Choose a base branch
from

Conversation

yeniatencio
Copy link
Contributor

@yeniatencio yeniatencio commented Sep 9, 2024

Ticket: https://digital-vic.atlassian.net/browse/SD-247
BE testing link:

Motive: We want to have a new media type and install private files Download permission module.

Config files.

@yeniatencio yeniatencio self-assigned this Sep 9, 2024
@MdNadimHossain
Copy link
Contributor

@yeniatencio I think it is best to make it as a submodule and move all these settings in there. Also making a install hook, so everything gets installed when the related module is installed. This way it will be easier to manage and turn it on and off for different projects. Also if possible create a uninstall hook just in case, if the module needs to be uninstalled, it will clear the related fields and settings.

Copy link
Contributor

@vincent-gao vincent-gao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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

Successfully merging this pull request may close these issues.

3 participants