refactor: to useCopyToClipboard to catch errors - when user has disable copy clipboard permission in browser

[Nabhag8848]

as discussed here #13292 (comment) with @prastoin

• introduce optional message param to copyToClipboard method in useCopyToClipboard and refactor it across app, as in case if user has disallowed clipboard permission in BROWSER unprotected clipboard access breaks without catch.

• Email copied to clipboard

• run lingui extract

[greptile-apps]

Greptile Summary

This PR introduces better error handling for clipboard operations by refactoring the useCopyToClipboard hook. The main changes:

1. Adds an optional message parameter to the copyToClipboard method to allow for context-specific success messages
2. Implements proper error handling for cases where users have disabled clipboard permissions in their browser
3. Consolidates clipboard operations across the application to use this centralized hook
4. Updates all locale files to remove the generic "Copied to clipboard!" message in favor of context-specific messages

The refactoring improves the user experience by providing better feedback when clipboard operations fail due to permission issues, while maintaining the existing functionality when permissions are granted.

Confidence score: 5/5

1. This PR is safe to merge as it adds defensive programming without changing core functionality
2. The changes are focused on error handling and user feedback, with no risk to existing clipboard operations
3. The files that need attention are:
   • packages/twenty-front/src/hooks/useCopyToClipboard.tsx (core functionality)
   • Components using direct clipboard API calls (to ensure they've been migrated to the hook)

_{41 files reviewed, 4 comments}

_{Edit Code Review Bot Settings | Greptile}

[github-actions]

🚀 Preview Environment Ready!

Your preview environment is available at: http://bore.pub:46880

This environment will automatically shut down when the PR is closed or after 5 hours.

[greptile-apps]

Greptile Summary

This PR addresses the missing Russian translations that resulted from the clipboard functionality refactor. The ru-RU.po file was completely emptied during the lingui extract process, which removed all Russian translations for the application. This is a critical issue that needs immediate attention as Russian users will see untranslated UI text throughout the application.

The refactor introduced new clipboard-related translation strings like 'Email copied to clipboard', 'Copied to clipboard', and 'Couldn't copy to clipboard' that require Russian translations. However, the extraction process appears to have cleared the entire Russian locale file instead of properly updating it with the new strings while preserving existing translations.

Confidence score: 1/5

1. This PR is not safe to merge as it will break the user experience for all Russian-speaking users
2. The confidence score is 1/5 because the Russian locale file is completely empty, which will cause missing translations across the entire application
3. The file that needs immediate attention is packages/twenty-front/src/locales/ru-RU.po - it needs to be restored with proper Russian translations for all application strings, not just the new clipboard-related ones

_{44 files reviewed, 1 comment}

_{Edit Code Review Bot Settings | Greptile}

[greptile-apps]

style: Korean translation is missing for the new 'Email copied to clipboard' message

[Nabhag8848]

ready for review !

edit: I see we have github action for translation catalog extraction, have made the changes accordingly

[prastoin]

Hey there, that's a very good refactor
Thanks a lot for the good work !

[prastoin]

Remark: default duration value might differ from 2000, but anw that's a good standardizatin to have here

[github-actions]

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