Skip to content

feat: allow for custom message for approval and rejection - #330

Merged
lukasdotcom merged 4 commits into
mainfrom
feat/message
Oct 6, 2025
Merged

feat: allow for custom message for approval and rejection#330
lukasdotcom merged 4 commits into
mainfrom
feat/message

Conversation

@lukasdotcom

Copy link
Copy Markdown
Member
image image

@lukasdotcom
lukasdotcom requested review from julien-nc and removed request for julien-nc September 22, 2025 15:19
@lukasdotcom
lukasdotcom force-pushed the feat/message branch 4 times, most recently from cfcbd0b to db8e458 Compare September 22, 2025 15:55
Signed-off-by: Lukas Schaefer <lukas@lschaefer.xyz>

@julien-nc julien-nc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The "newMessage" in the "approved" event of is used by but not by .
So putting a reason in the modal does not work (i mean it's ignored).

Otherwise 👍

Signed-off-by: Lukas Schaefer <lukas@lschaefer.xyz>
Signed-off-by: Lukas Schaefer <lukas@lschaefer.xyz>

@julien-nc julien-nc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There is no more approve or reject file action for a pending file. It would be nice to have a "Approve or reject" action that opens the modal, just like when clicking the inline action.

Signed-off-by: Lukas Schaefer <lukas@lschaefer.xyz>
@lukasdotcom

Copy link
Copy Markdown
Member Author

There is no more approve or reject file action for a pending file. It would be nice to have a "Approve or reject" action that opens the modal, just like when clicking the inline action.

Added it:

image

@lukasdotcom
lukasdotcom requested a review from julien-nc October 1, 2025 13:44

@julien-nc julien-nc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Tested, works like a charm 👍 Thanks for adding the unified non-inline action.

@lukasdotcom
lukasdotcom merged commit eb7f34c into main Oct 6, 2025
40 checks passed
@lukasdotcom
lukasdotcom deleted the feat/message branch October 6, 2025 12:26
@lukasdotcom lukasdotcom mentioned this pull request Oct 6, 2025
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