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

Feat: support relative media paths via config #193

Merged
merged 1 commit into from
Nov 2, 2023

Conversation

awcodes
Copy link
Owner

@awcodes awcodes commented Nov 2, 2023

No description provided.

@awcodes awcodes merged commit 18a0355 into 3.x Nov 2, 2023
5 checks passed
@robin-dongbin
Copy link

So you realize the problem yourself now?
Just opinionated.
#133

@awcodes
Copy link
Owner Author

awcodes commented Nov 3, 2023

This has nothing to do with your previous PR. But if it makes you feel better about yourself then sure you were right.

@robin-dongbin
Copy link

My previous PR is because I want to use relative media path
#133 (comment)

@awcodes
Copy link
Owner Author

awcodes commented Nov 3, 2023

I'm not going to do this with you. You're PR didn't solve the issue and was therefore not merged. You can now use your relative urls. Please feel free to write you're own plugin and do what ever you want with it. But there is zero need to bring your closed Pr into this at this time.

@Nestoran
Copy link

Would you please implement this relative path support into v2?

@awcodes
Copy link
Owner Author

awcodes commented Jan 19, 2024

I'm open to a PR for it, but have no plans to back port features into previous versions.

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.

3 participants