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

Nova 4 Update #89

Open
wants to merge 23 commits into
base: master
Choose a base branch
from
Open

Conversation

ali-raza-saleem
Copy link

@ali-raza-saleem ali-raza-saleem commented May 11, 2022

-except Trix and Callback functionality

@muhammadsaeedparacha
Copy link

Tested, working as expected.
Can someone merge this please

@classic-o
Copy link
Owner

classic-o commented Jun 13, 2022

Sorry. Currently I have not time for this library and for reading about Nova 4. I tried to run your changes and have a lot of problems in UI and while uploading file.
1
2

If you dont want to wait me, will be better to do fork for Nova 4...

@ali-raza-saleem
Copy link
Author

@classic-o I also updated the UI. It's issue on Nova's side. Some tailwind classes used in the package don't work because Nova's tailwind implements tree shaking.
Temporary Fix: laravel/nova-issues#373 (comment)

@classic-o
Copy link
Owner

@the-alichemist see the 2nd screenshot, component in entity does not work.
Write to me (to my email address) your telegram or another way to connect with you. Maybe our work will go faster

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