Skip to content

fix: createContentLoader generates invalid url when sets srcDir #2578

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

Merged
merged 3 commits into from
Jul 26, 2023

Conversation

zonemeen
Copy link
Collaborator

@zonemeen zonemeen commented Jul 4, 2023

closes #2540
closes #2462

@brc-dd
Copy link
Member

brc-dd commented Jul 5, 2023

I'll get back on this. Need to check on some projects as this will be a breaking change. Seems right though.

@brc-dd brc-dd self-assigned this Jul 5, 2023
@SapneshNaik
Copy link

SapneshNaik commented Jul 16, 2023

@brc-dd, when will this be merged? This is causing a weird issue where the logo is no longer showing if I set a custom srcDir rather leaving it blank as it comes by default.

@brc-dd
Copy link
Member

brc-dd commented Jul 17, 2023

@SapneshNaik What does this have to do with logo?

@SapneshNaik
Copy link

@brc-dd the Logo URL (set in the themeconfig ) breaks if you set a different source dir using the srcDir config option.

@brc-dd
Copy link
Member

brc-dd commented Jul 18, 2023

@SapneshNaik That isn't related to this PR. Please create a separate issue for that.

@domnantas
Copy link
Contributor

Agree that this change makes sense, I don't see any reason for createContentLoader to have access outside srcDir. Docs would need to be updated here too https://vitepress.dev/guide/data-loading#createcontentloader

@brc-dd brc-dd merged commit 74d9ba2 into vuejs:main Jul 26, 2023
@zonemeen zonemeen deleted the fix/contentLoader branch July 26, 2023 08:47
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants