Skip to content

draft: add browser revive services menu#448

Draft
AlexTzib wants to merge 1 commit into
Manuito83:developfrom
AlexTzib:draft/browser-revive-services
Draft

draft: add browser revive services menu#448
AlexTzib wants to merge 1 commit into
Manuito83:developfrom
AlexTzib:draft/browser-revive-services

Conversation

@AlexTzib
Copy link
Copy Markdown
Contributor

@AlexTzib AlexTzib commented May 5, 2026

Draft PR for #90 so we can discuss the direction before polishing this further.

Hi @Manuito83, I wanted to check whether this matches what you had in mind for adding revive services to the browser.

What this draft tries:

  • Adds a small revive-services menu icon to the browser app bar when the cached user status says the player is in Hospital.
  • Reuses the existing enabled revive-service settings, so only active providers appear.
  • Reuses the existing revive dialogs for Nuke, UHC, WTF, Midnight X, The Wolverines, and Combat Ready.
  • Passes no profile object to those dialogs, so they fetch fresh profile data before sending a request. That keeps the existing safety check that blocks the request if Torn says the user is not hospitalized.

Why this approach:
I avoided injecting buttons into Torn's hospital/profile DOM because that would be more fragile. This keeps the feature inside Torn PDA's browser chrome and reuses the revive flows that already work from Profile/War.

Questions before this becomes a normal PR:

  • Is the browser app bar the right place for this, or would you prefer the vertical tab/menu area or only the hospital page?
  • Should the icon appear only when cached status says Hospital, or should it always be available and let the dialog do the fresh status check?
  • Should HeLa stay omitted here, matching the current commented-out profile/war usage, or should it be included too?

Local verification:

  • Ran git diff --check.

I could not run Flutter analyzer/build locally because flutter is not available on this PATH, so CI is the compile check.

@Manuito83
Copy link
Copy Markdown
Owner

Manuito83 commented May 24, 2026

Perhaps we could do as we do when we attack someone? Give the page title frame a noticeable colour (in case they use the dialog browser style, if I recall correctly, it's a word just below the CLOSE button)... and then have a button for this where the "REQUEST ASSISTANCE" (o similar) button is now?

I'm just trying to avoid filling the app bar too much, as in certain pages it's already at the limit!

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