Skip to content
This repository was archived by the owner on Sep 25, 2025. It is now read-only.

Conversation

@David-Pena
Copy link
Collaborator

I'm not used to setup aliases but I tried to follow what was already done in the project... some of the relative path replacements work as expected but these three files have imports which I have trouble with:

  • playground/src/main.ts
  • .vitepress/theme/components/Changelog.vue
  • .vitepress/theme/components/SidebarNavBefore.vue

I added // eslint-disable-next-line no-restricted-imports in those files (It's not the goal)

@Barbapapazes I'd appreciate some guidance in this if possible 🙏🏻🙏🏻

@David-Pena David-Pena added the enhancement New feature or request label Feb 3, 2025
@David-Pena David-Pena self-assigned this Feb 3, 2025
@Barbapapazes
Copy link
Contributor

Hey 👋,

thanks for trying to solve this issue! When I was saying "everywhere" it was only in the src directory. There is no real need to solve this in .vitepress and playground. In the src directory, it's very important to have our tools working since they try to detect @/ui paths.

@David-Pena
Copy link
Collaborator Author

thanks for trying to solve this issue! When I was saying "everywhere" it was only in the src directory.

Ohh I'll make sure only src is covered

@Barbapapazes
Copy link
Contributor

thanks for trying to solve this issue! When I was saying "everywhere" it was only in the src directory.

Ohh I'll make sure only src is covered

Thanks! Sorry, I should have explained the issue better

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Feb 4, 2025

Deploying vue-ui with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9c3a138
Status:🚫  Build failed.

View logs

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Feb 4, 2025

Deploying vue-ui-playground with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9c3a138
Status: ✅  Deploy successful!
Preview URL: https://82f4bf6b.vue-ui-playground.pages.dev
Branch Preview URL: https://david-feature-130-disallow-r.vue-ui-playground.pages.dev

View logs

@David-Pena
Copy link
Collaborator Author

@Barbapapazes

I reverted the changes related to relative paths on vitepress, removed the comment and I think the PR is ready to be reviewed

@Barbapapazes Barbapapazes changed the title feat: disallow relative path everywhere chore: disallow relative path everywhere Feb 12, 2025
@Barbapapazes Barbapapazes merged commit 74566d4 into main Feb 12, 2025
4 of 5 checks passed
@Barbapapazes Barbapapazes deleted the david/feature/130-disallow-relative-path-everywhere branch February 12, 2025 11:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants