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

Fixes file upload for Filament 3 #28

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

adampatterson
Copy link

Replaces getComponentFileAttachmentUrl with getFormComponentFileAttachmentUrl

I think this will also resolve #25 #27

Replaces `getComponentFileAttachmentUrl` with `getFormComponentFileAttachmentUrl`
Copy link

what-the-diff bot commented Sep 8, 2024

PR Summary

  • Update of method name in editor.js
    The code modification in editor.js is about a change in the name of a method we use. Previously, we used the getComponentFileAttachmentUrl method, but now we've transitioned to using the getFormComponentFileAttachmentUrl method. This adjustment helps us stay more consistent in identifying the exact purpose of the method across different parts of the application.

  • Addition of a logging statement in home.blade.php
    The adjustment made in home.blade.php includes adding a log statement intended to deliver a message every time a file is uploaded. This log message, "File uploaded!", and the uploaded file's name will give us immediate feedback when an upload occurs, assisting in diagnosing any issues related to file uploads. This will improve our capability to monitor and inspect the upload process in real-time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant