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

Changes in request and logic saving the files #19

Closed
wants to merge 5 commits into from

Conversation

dacastro4
Copy link

Route 404 issue

I fixed the error that I was getting (#18) when trying to process a file.
I had to do this to fix it

Route::post('/process/{resource}/{resourceId}', [ FilepondController::class, 'process' ])->name('nova.filepond.process');

It was an issue with the NovaRequest class which required a "resource" in the URL.

JSON column value

I had to make a couple of changes in here to able to do the normal process with JSON column from DB. If you need an example let me know.

Removing file after moving

I'm removing the file after it's moved from the temp folder here

temp location and naming convention

So I decided to add a better naming convention using the file's name because it was really confusing with the random names.
Also, I'm using the storage folder for temp files instead of the system's tmp folder.
code

Files inside tabs

I made a couple of changes that allow inputs inside the Tab class to use this custom Field
code

Vue Index Field

I added a simple list of file names so at least we can see the file names <- (WIP)

@milewski
Copy link
Member

I see, so the 404 error was caused by https://github.com/dcasia/nova-filepond/blob/master/src/Http/Controllers/FilepondController.php#L130 $resource::newModel() couldn't find the model because missing the resource on the URL, this must have been a change on latest nova versions before it wasn't required.

Regardless thanks for the PR let me know when it's ready to merge

@AlexAzartsev
Copy link

AlexAzartsev commented Jun 30, 2020

Hello, I faced the same issue with 404 process url, will this fix be merged into master?

@dacastro4 dacastro4 closed this Mar 29, 2021
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.

None yet

3 participants