#Tagging OP in Discord when posting reply to issue

74 messages · Page 1 of 1 (latest)

midnight monolith
#

I've had a look but I can't see an option to enable tagging the OP in Discord when replying to their issue, like it does when a request becomes available. Is there an option I'm missing, or is this something that needs to be submitted as a feature request?

#

Related, though happy to open a second support ticket if need be, the "Manage Issues" permission node. Does that allow anyone to manage any issues, or just theirs? I'd like for users to be able to close their own issues if it has been resolved.

sour merlin
sour merlin
midnight monolith
midnight monolith
midnight monolith
sour merlin
#

The ability for users to close their own issues does sound useful 🤔

midnight monolith
sour merlin
midnight monolith
sour merlin
midnight monolith
sour merlin
#

You can close it and keep commenting on it - both regular users and admins

#

But once it's closed, only someone with "Manage Issues" perms or higher can re-open it

midnight monolith
#

Ah. So managing own issues would be beneficial in more than one way. 🤔

sour merlin
#

Yep yep, kind of how users can delete their own request before it's approved if they change their mind

midnight monolith
#

@sour merlin did you want me to open a bug report for the no tagging as well while I'm at it?

sour merlin
#

Sorry wasn't able to get back to Discord until now. Looks like you opened one, thank you!

midnight monolith
#

All good. I opened one for the permissions node, but not for the tagging. I'll do the tagging one shortly. ☺️

sour merlin
#

I'm gonna go check it out in bit, let me do that first then I'll let you know if it's needed

midnight monolith
#

Oh sweet! No worries. 😁

modern chasm
#

You screenshotted the system Discord notifications page, which allows you to customize which notifications are sent. Users can then configure their own preferences for when they are mentioned in their user settings.

midnight monolith
midnight monolith
modern chasm
midnight monolith
modern chasm
#

Hm, interesting. If the issue OP isn't getting tagged when others comment on their issue, then it's a bug. Added this functionality long ago and haven't looked at it in ages...

midnight monolith
#

(Removed the User ID)

modern chasm
#

might want to crop your screenshot as well, shows your domain in the url bar

midnight monolith
midnight monolith
modern chasm
#

can you also screenshot the user's permissions? i'm taking a look and wondering about something...

#

(do they have the manage issues perm?)

#

(or did they when the notification was triggered?)

midnight monolith
#

Alright, on my laptop. Lemme have a look.

midnight monolith
#

(Same user)

modern chasm
#

sus_think hmm

midnight monolith
#

Happy to have a play with my test user if you'd like me to switch perms on and off.

modern chasm
#

Just to confirm, you are using the Discord agent and not the webhook notification agent?

midnight monolith
#

I assume yes? 😹

modern chasm
#

I mean this was tested before when I introduced it over a year ago

#

😅

midnight monolith
#

I promise I didn't break anything! 😹

modern chasm
midnight monolith
#

I've still got that bug report on Github ready to lodge, if you'd like it. 😹

modern chasm
#

I'm going to test it out, just need to spin up my dev env

midnight monolith
modern chasm
#

Lol I actually had my local repo get corrupted some time ago and haven't been working on Overseerr much since kao_lazy But hm, I'm reproing this. Wonder why

midnight monolith
#

Urgh. That's always annoying.

But we appreciate the effort! cat_pray

modern chasm
#

Ah found the issue. Will open a PR with a fix. Maybe broke when typeorm was updated maybe? kao_think

midnight monolith
#

Awwww yiss!!! This is why they pay you the big bucks! 😹

#

(And if they don't, demand a raise! 😁 )

modern chasm
#

I'll also add a fix for users not being able to open/close their own issues while I'm at it.

midnight monolith
#

Demand double! 😹

modern chasm
#

haven't touched the overseerr codebase in ages, I don't even remember writing some of this code lol

#

some of the notifs stuff was def me

midnight monolith
midnight monolith
modern chasm
#

well, I didn't bump the dependencies cktongue

modern chasm
#

Did you submit the issue on github? I'll tag it in my PR

midnight monolith
#

Just did.

#

3266

modern chasm
midnight monolith
#

Yis!

#

Thank you!

#

"Issues with issues". Chuckles

modern chasm
midnight monolith
midnight monolith
#

@modern chasm so one last random question: any idea when the build will be available to update? 😹