Add useUploadAttachment hook (#7617)

[yadavshubham01]

Reusing the useUploadAttachment Hook

In the implementation of the feature to ensure the attachment table is updated whenever new images are added to a RICH_TEXT field #7617 , it is likely that the useUploadAttachment hook is reused.

The useUploadAttachment hook is responsible for handling the upload of attachments, including images, and returning the uploaded file URL. By reusing this hook, you can leverage its existing functionality to handle image uploads within the RICH_TEXT field.

In this case, the modified image handling logic would utilize the useUploadAttachment hook to upload new images added to the RICH_TEXT content. The hook would then return the uploaded file URL, which would be used to update the attachment table with the details of the newly added images.

By reusing the useUploadAttachment hook, you can avoid duplicating code and ensure consistency in the way attachments are handled throughout the application.

Fixes #6565

[greptile-apps]

PR Summary

This pull request introduces a new useUploadAttachment hook and integrates it into the RichTextEditor component to handle image uploads and update the attachment table. Here's a summary of the key changes:

• Added useUploadAttachment hook in packages/twenty-front/src/modules/activities/files/hooks/useUploadAttachment.tsx for file uploads and attachment creation
• Modified RichTextEditor component to use the new hook for image uploads
• Updated GraphQL operations in packages/twenty-front/src/modules/attachments/graphql/queries/uploadFile.ts to support file uploads and attachment creation
• Added AddAttachment mutation in packages/twenty-front/src/generated/graphql.tsx for inserting attachment details

Key points to note:

• The useUploadAttachment hook combines file upload and attachment creation in a single operation
• RichTextEditor now uses the new hook instead of separate upload and attachment mutations
• The implementation in RichTextEditor needs refinement, as there are some inconsistencies with the new hook usage
• Error handling in the new hook could be improved for better robustness

_{4 file(s) reviewed, 6 comment(s)
Edit PR Review Bot Settings | Greptile}

[yadavshubham01]

@charlesBochet please review this PR

[charlesBochet]

This PR does not look functional

[yadavshubham01]

there any issue in this PR @charlesBochet

[charlesBochet]

/award 300

[oss-gg]

Awarding yadavshubham01: 300 points 🕹️ Well done! Check out your new contribution on oss.gg/yadavshubham01

[lucasbordeau]

Follow-up issue : #8351

[github-actions]

Thanks @yadavshubham01 for your contribution!
This marks your 4th PR on the repo. You're top 8% of all our contributors 🎉
See contributor page - Share on LinkedIn - Share on Twitter