Bug 458911 - Add ability to enter (optional) deletion reason
Summary: Add ability to enter (optional) deletion reason
Status: RESOLVED FIXED
Alias: None
Product: NeoChat
Classification: Applications
Component: General (other bugs)
Version First Reported In: unspecified
Platform: Other Linux
: NOR wishlist
Target Milestone: ---
Assignee: Tobias Fella
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-09-09 07:35 UTC by Justin Zobel
Modified: 2022-11-18 00:24 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Justin Zobel 2022-09-09 07:35:17 UTC
Add the ability to add a message if desired when deleting messages.

Maybe `Remove (with reason)` as a context menu option.
Comment 1 Jan Bidler 2022-09-11 11:37:49 UTC
Sounds similar to BUG: 457473 for me.

I don't think that another context menu is worth it, just prompting an additional optional text box during delete/kicking/banning etc.
Comment 2 Justin Zobel 2022-09-11 11:55:48 UTC
Provided it's an optional tickbox to enter the message that's fine.

On 11/9/22 21:07, Jan Bidler wrote:
> https://bugs.kde.org/show_bug.cgi?id=458911
>
> Jan Bidler <janbidler00@protonmail.com> changed:
>
>             What    |Removed                     |Added
> ----------------------------------------------------------------------------
>                   CC|                            |janbidler00@protonmail.com
>
> --- Comment #1 from Jan Bidler <janbidler00@protonmail.com> ---
> Sounds similar to BUG: 457473 for me.
>
> I don't think that another context menu is worth it, just prompting an
> additional optional text box during delete/kicking/banning etc.
>
Comment 3 Bug Janitor Service 2022-11-17 23:56:46 UTC
A possibly relevant merge request was started @ https://invent.kde.org/network/neochat/-/merge_requests/636
Comment 4 Tobias Fella 2022-11-18 00:24:49 UTC
Git commit d70f77f6225a7518a81802e8149490168172cb91 by Tobias Fella.
Committed on 17/11/2022 at 23:56.
Pushed by tfella into branch 'master'.

Allow adding a reason for deleting messages

M  +4    -4    src/neochatroom.cpp
M  +2    -2    src/neochatroom.h
M  +6    -2    src/qml/Dialog/UserDetailDialog.qml
M  +4    -4    src/qml/Menu/Timeline/FileDelegateContextMenu.qml
M  +4    -1    src/qml/Menu/Timeline/MessageDelegateContextMenu.qml
A  +53   -0    src/qml/Menu/Timeline/RemoveSheet.qml     [License: GPL(v2.0+)]
M  +1    -0    src/res.qrc

https://invent.kde.org/network/neochat/commit/d70f77f6225a7518a81802e8149490168172cb91