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

Update with stable changes #3

Closed
wants to merge 69 commits into from
Closed

Update with stable changes #3

wants to merge 69 commits into from

Conversation

codingneko
Copy link
Owner

What

Why

Additional info (optional)

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

codingneko and others added 30 commits October 2, 2023 11:29
This reverts commit e0fd136.
This reverts commit e0fd136.
Fixes an issue where if approval and invite were enabled the code would stay as unused
Mar0xy and others added 25 commits October 21, 2023 12:40
* the previous one could allow a SQL injection, since the
  `opts.filetype` value came straight from the browser

* this more precise regex match will not produce spurious
  matches (which were very unlikely, true, but still, let's be
  precise) (`video/movingimages` would have matched `%image%`!)
intended to allow easier creation of custom CSS modifications. the last
4 chars of class names should be static as long as files don't move, but
just in case modifications might prefer to use prefix selectors such as
[class^=MkWhatever] instead of .MkWhatever-1234
@codingneko
Copy link
Owner Author

wrong branches damnit

@codingneko codingneko closed this Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants