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

fix: 一部の状態のファイルをドロップしてアップロードできない場合がある問題を修正 (投稿フォーム以外についても) #9114

Merged
merged 2 commits into from
Oct 12, 2022

Conversation

m-hayabusa
Copy link
Contributor

What

#9035 で変更した箇所に漏れがあった: ドライブ、メッセージのファイルをドロップできる部分についても同様に修正した

具体的には、effectAllowedall でないときにも基本的に copy を使用するようにし、 move しか受けつけない条件でのみ move を使用するようにした (#9035 と同じ内容) 。

Why

例えばアプリケーションから直接書き出されたファイルのような、「移動することはできないがコピーすることができる」状態のファイルがドロップされた場合、 move 操作は拒否され、結果として、ファイルはアップロードされない。

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Oct 3, 2022
@syuilo syuilo merged commit dae8251 into misskey-dev:develop Oct 12, 2022
@syuilo
Copy link
Member

syuilo commented Oct 12, 2022

🙏

Jeder321 pushed a commit to Jeder321/misskey that referenced this pull request Nov 21, 2022
* Fixed association between dropEffect and effectAllowed as well as the submission form
* Fixed that strings can be dropped

Reviewed-on: misskey-dev#9114
Changelog: Fixed
yu256 pushed a commit to yu256/akatsukey that referenced this pull request Mar 5, 2023
…ev#9114)

* dropEffectとeffectAllowedの関連付けを投稿フォームと同様に修正

* 文字列をDropできるようにしてしまったのを修正
@m-hayabusa m-hayabusa deleted the fix-client-drag-drop-files branch December 18, 2023 11:18
morotesovtannu pushed a commit to ganngi-mari/missnemone that referenced this pull request Jan 20, 2024
…ev#9114)

* dropEffectとeffectAllowedの関連付けを投稿フォームと同様に修正

* 文字列をDropできるようにしてしまったのを修正
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants