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

Fix: restore project dependencies & environment setup info #380

Open
funder7 opened this issue Nov 11, 2024 · 0 comments
Open

Fix: restore project dependencies & environment setup info #380

funder7 opened this issue Nov 11, 2024 · 0 comments

Comments

@funder7
Copy link

funder7 commented Nov 11, 2024

Hi,

I've found your project in the awsome-vue page, curious to see an app made with Naive UI, but I wasn't able to obtain a working build, after following the setup instructions.

After a little research, I've found a problem related with two libraries & a custom eslint config.

Now, I'm not an expert with pnpm, but I think it still uses npm packages, and these libs you made are not available in npmjs repositories:

  • @dolphin-admin/utils
  • @dolphin-admin/auto-import
  • @dolphin-admin/bootstrap-animation

So I cloned your repo and used this syntax in the package.json to load them from the filesystem:

"@dolphin-admin/utils": "link:../core/packages/utils",

Unfortunately even that repository had some inconsistencies (check my PR there: dolphin-admin/core#22), after fixing them, I was finally able to run this project.

I think that you need to add some info in the README, as a noob like me doesn't know your project, and wants the soup ready & cooked 😃 - like the APIs, why don't you link the https://github.com/dolphin-admin/nest-api ?

Hopefully this doesn't sound like a critique, but it looks like you spent some time to develop this, why don't you complete the missing 5% to honour your hard work?!

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

No branches or pull requests

1 participant