fix: remove modal when navigating back

[mohit-nagaraj]

⚠️⚠️⚠️ Since we do not accept all types of pull requests and do not want to waste your time. Please be sure that you have read pull request rules:
https://github.com/louislam/uptime-kuma/blob/master/CONTRIBUTING.md#can-i-create-a-pull-request-for-uptime-kuma

Tick the checkbox if you understand [x]:

• I have read and understand the pull request rules.

Description

Fixes #5008

Type of change

Please delete any options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)
• User interface (UI)

Checklist

• My code follows the style guidelines of this project
• I ran ESLint and other linters for modified files
• I have performed a self-review of my own code and tested it
• I have commented my code, particularly in hard-to-understand areas (including JSDoc for methods)
• My changes generates no new warnings
• My code needed automated testing. I have added them (this is optional task)

Screenshots (if any)

Uptime.Kuma.-.Google.Chrome.2024-10-08.08-27-34.mp4

[mohit-nagaraj]

Do let me kno if other pop ups also needs these life cycle events to be implemented

[CommanderStorm]

Thanks for the PR,

I have left a few questions below, mostly regarding a bit of likely code-duplication.

[louislam]

Feel too hacky and unsafe, as it's utilized dom and added a lot of events. I prefer not to change the current logic.

[mohit-nagaraj]

Feel too hacky and unsafe, as it's utilized dom and added a lot of events. I prefer not to change the current logic.

hey i have updated the code now, does this make it better.... i am just now relying on beforeUnmount hook to remove the modal

[CommanderStorm]

Please remove the unrelated changes.
Otherwise this looks good to me

[mohit-nagaraj]

Done removed those changes

[mohit-nagaraj]

@CommanderStorm when is the earliest possible time u can retake a look in this again