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

Bump max supported version. #846

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Bump max supported version. #846

wants to merge 1 commit into from

Conversation

thw26
Copy link
Contributor

@thw26 thw26 commented Mar 5, 2024

I submitted a similar PR for 2.15.1. See #742. Various tests were added at that time by @icewind1991. I tried to review those changes to see if there was anything I would need to add but I did not see anything and could not tell otherwise. I had copied the original PR from a prior version bump commit, and am simply doing the same here. I am just trying to get the dashboard not to complain about it being incompatible.

EDIT: As mentioned in icewind1991/files_markdown#219, the app is no longer opening there to edit the file. Instead markdown files are downloaded directly. Testing this, this also downloads the text file immediately rather than opening the editor. I will have to set up a dev environment to debug this further. This will not be as simple as #742.

@thw26 thw26 marked this pull request as draft March 5, 2024 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant