Skip to content

Copy log fixes - #840

Merged
icewind1991 merged 4 commits into
masterfrom
copy-fixes
Mar 1, 2023
Merged

Copy log fixes#840
icewind1991 merged 4 commits into
masterfrom
copy-fixes

Conversation

@icewind1991

Copy link
Copy Markdown
Member
  • When importing logs from clipboard/files, clicking on the "copy" menu would open the menu for every item because the id field was unset
  • "Copy formatted" didn't include the previous/cause exceptions

Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991

Copy link
Copy Markdown
MemberAuthor

/compile amend

Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@icewind1991
icewind1991 merged commit a1a2c9d into masterMar 1, 2023
@delete-merged-branch
delete-merged-branchBot deleted the copy-fixes branch March 1, 2023 17:44
@blizzzblizzz mentioned this pull request Mar 2, 2023
@joshtrichards

Copy link
Copy Markdown
Member
  • "Copy formatted" didn't include the previous/cause exceptions

Was about to fix this because I've come across it a few times, then discovered it's already fixed! Woohoo! Guess it's only been <v26 instances I've seen this happen on.

Should we consider backporting this to v25? Users often send in formatted snippets in their issues.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@icewind1991@joshtrichards@juliusknorr