Skip to content

feat(card): upload images pasted from the clipboard as attachments#8042

Open
SadeqRouhani wants to merge 1 commit into
nextcloud:mainfrom
SadeqRouhani:feat/paste-image-attachments
Open

feat(card): upload images pasted from the clipboard as attachments#8042
SadeqRouhani wants to merge 1 commit into
nextcloud:mainfrom
SadeqRouhani:feat/paste-image-attachments

Conversation

@SadeqRouhani

Copy link
Copy Markdown
Contributor
  • Resolves: #
  • Target version: main

Summary

TODO

  • ...

Checklist

  • Code is properly formatted
  • Sign-off message is added to all commits
  • Tests (unit, integration, api and/or acceptance) are included
  • Documentation (manuals or wiki) has been updated or is not required

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Signed-off-by: sadegh <sa.rohani72@gmail.com>
const extension = extensions[file.type] ?? (file.type.split('/')[1] ?? 'png')
const d = new Date()
const pad = (n) => String(n).padStart(2, '0')
const stamp = `${d.getFullYear()}${pad(d.getMonth() + 1)}${pad(d.getDate())}-${pad(d.getHours())}${pad(d.getMinutes())}${pad(d.getSeconds())}`

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To ignore duplicate, it should be

const stamp = `${d.getFullYear()}${pad(d.getMonth() + 1)}${pad(d.getDate())}-${d.getTime()}`

'image/bmp': 'bmp',
'image/svg+xml': 'svg',
}
const extension = extensions[file.type] ?? (file.type.split('/')[1] ?? 'png')

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should extract file extension from file name instead

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.

2 participants