Skip to content
This repository was archived by the owner on Feb 15, 2026. It is now read-only.

feat: Add optional message on declining media request#3714

Open
KyleKincer wants to merge 2 commits intosct:developfrom
KyleKincer:feature-deny-message
Open

feat: Add optional message on declining media request#3714
KyleKincer wants to merge 2 commits intosct:developfrom
KyleKincer:feature-deny-message

Conversation

@KyleKincer
Copy link

Description

This change adds a modal to allow an admin user to add an optional message to the user when declining a media request. This message is included in any notification to the user of their declined request. The decline text is also visible on the request itself.

Screenshot (if UI-related)

image image

To-Dos

  • Implement on RequestCard component-- This only works from the Requests page currently.
  • Implement for the rest of the notification services. I don't currently have a way to test all of them. It's implemented for Discord (tested), and Email and WebPush (untested)

Issues Fixed or Closed

@KyleKincer KyleKincer changed the title Add optional message on declining media request feat: Add optional message on declining media request Dec 15, 2023
@stale

This comment was marked as outdated.

Copy link

@Richardvaio Richardvaio left a comment

Choose a reason for hiding this comment

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

please let me know if ive give enough information. im new to this

Choose a reason for hiding this comment

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

ALTER TABLE media_requests - these need the "s" removed = ALTER TABLE media_request


const [isRetrying, setRetrying] = useState(false);

const declineRequest = async (declineMessage: string) => {

Choose a reason for hiding this comment

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

Replacing a few parts to include Declined Reason before the decline message in the list.

  • Replace this
    {requestData.status === MediaRequestStatus.DECLINED && requestData.adminMessage && (



    {intl.formatMessage(messages.declinedreason)}


    {requestData.adminMessage.replace(/^"|"$/g, '')}


    )}

  • Add this
    const messages = defineMessages({
    declinedreason: 'Declined Reason',

  • and add into i18n\local
    "components.RequestList.RequestItem.declinedreason": "Declined Reason",

Choose a reason for hiding this comment

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

line 390 to 399

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ability to add optional message when denying a request

3 participants