search

icehouse-ventures / laravel-chartjs

Simple package to facilitate and automate the use of charts in Laravel using the Chartjs library
https://icehouse-ventures.github.io/laravel-chartjs/
MIT License
65 stars 7 forks source link

is package incomplete? #21

Closed nirvale closed 6 days ago

nirvale commented 1 month ago

Hello, I did receive error response ...

imagen

And the src files...

imagen

imagen

And at exports config ...

imagen

What I'm doing wrong? I did install with composer and try packagist and github repo...

JayChurchward commented 2 weeks ago

Also having the exact same problem. Installed via composer, the publish command can't find the publish file, and the classes aren't found anywhere.

image

JayChurchward commented 2 weeks ago

@nirvale Try removing the version you installed an reinstall it with composer require icehouse-ventures/laravel-chartjs ^1.4

Without specifying the version I was getting a 3.0.0 version installed which didn't match the repo at all and still had namespaces referring to fxcosta. After I specified the version I was then able to publish the provider.

peterthomson commented 2 weeks ago

You can use the old syntax app()->chartjs->name() if the new Charts::name() (with the imported Facade) isn't working. But the version weirdness, and package asset publishing issues sounds like something to do with the package version tagging or the install method. The folder structure in your screenshots looks like the old folder structure of the package (notice the absence of the Facade folder and Support folder (which are in the latest version). Maybe try composer requiring a specific version (1.4 is the latest) and see how that goes?

peterthomson commented 6 days ago

Ok, we found the bug. This package was forked from a prior package. And somehow Packagist has now decided that they are the same package, so whenever the deprecated package makes a version bump that becomes the 'latest' version. There are a number of long-term fixes such as detaching the fork, but the fastest fix was to bump the version on this package. I've updated the version numbering so any future versions of the package should be less likely to get derailed.