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

Improved markdown support for companies #1785

Conversation

tomudding
Copy link
Member

Description editors in the company module now use a special Markdown editor tailored to companies. This means that it is possible to upload images and insert videos. To make this possible, several CommonMark extensions have been added to our setup.

This also separates uploads per companies and ensures that images are not rendered in non-company markdown views.

image

Closes GH-1781, fixes GH-1780, closes GH-1759, and closes GH-1576.

Utilises the new MarkdownCompanyEditor that also allows photos to
be uploaded and videos to be inserted.

Files from each company are now stored in a company specific folder
to prevent these files from overwriting other files (either those
from GEWIS itself or other companies).

IMPORTANT: files are stored based on the ID of the company not the
slug. This is because the slug can change and the ID should not
change.
Prevents image injection into Markdown when we do not want to show
images. For example, in activity descriptions or new items. However,
for companies this is useful. The images go through Glide!

Furthermore, a new renderer has been added that can render YouTube
and Vimeo videos.
@tomudding tomudding force-pushed the feature/improved-markdown-support-for-companies branch from c0022d3 to 86af1b4 Compare February 6, 2024 23:00
@tomudding tomudding merged commit 637a3da into GEWIS:main Feb 6, 2024
4 checks passed
github-actions bot added a commit that referenced this pull request Feb 6, 2024
Tom Udding: Merge pull request #1785 from tomudding/feature/improved-markdown-support-for-companies

Improved markdown support for companies

Co-authored-by: tomudding <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant