torn-pda icon indicating copy to clipboard operation
torn-pda copied to clipboard

Enable retaliation alerts without enabling notifications

Open danielbarnes175 opened this issue 1 year ago • 3 comments

Summary

Currently, for retaliation alerts to work, at least one user in your faction with faction API permissions has to have the alert enabled. After they have enabled it, then users without faction API access are able to use it. This, however, can be a problem if the user with faction API permissions wants to let other faction members use the alert, but they themselves do not want these notifications.

Request

  • Add a toggle for those with faction API permissions to be able to "enable" retaliation alerts without actually turning on the alert on their device.

Currently the UI looks like this: image We can have two options:

  • Retaliation Description: Get notified whenever it is possible to initiate a retaliation attack. Someone with Faction API access in your faction must have retaliation for others enabled.
  • Retaliation (For Others) Description: Allow faction members without Faction API access to use retaliation alerts. This will not have any effect if you do not have faction API access

Then just also make it so that if you turn on Retaliation then it will also check the box for Retaliation (For Others)

danielbarnes175 avatar Feb 15 '24 20:02 danielbarnes175

Thanks!

Manuito83 avatar Feb 15 '24 21:02 Manuito83

@danielbarnes175 would this make sense?

Image

Image

Manuito83 avatar Feb 15 '24 22:02 Manuito83

Would this additional toggle only show for users with faction API permissions? There’s nothing that says it won’t have any effect if people don’t have faction API access, and I think that’s important.

I also think …might be used by the server to send notifications about retaliations to all other members can be more succinct with …send retaliation notifications to faction members but this feels more of a personal preference of faction members over all other members so I’m fine with either.

Lastly, there’s a couple minor typos Relation alerts -> Retaliation alerts

Other than that, I think it’s good!

danielbarnes175 avatar Feb 16 '24 02:02 danielbarnes175

Thanks again!

Would this additional toggle only show for users with faction API permissions? There’s nothing that says it won’t have any effect if people don’t have faction API access, and I think that’s important.

Yes, this option will only appear if Torn PDA detects that the user has got Faction API privileges.

Manuito83 avatar Feb 16 '24 16:02 Manuito83

This has been merged to the beta branch and will make it to v3.3.0.

Firebase functions have already been updated.

Manuito83 avatar Feb 16 '24 17:02 Manuito83

Thanks for quickly addressing this!

danielbarnes175 avatar Feb 16 '24 18:02 danielbarnes175