Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

loader.js: Don't refresh twice upon __CLOSE__ #5106

Merged
merged 2 commits into from
Sep 7, 2023

Conversation

nilmerg
Copy link
Member

@nilmerg nilmerg commented Sep 6, 2023

This has been hardcoded as X-Icinga-Refresh didn't have any effect. With this change, it has. So to still double refresh, transmit X-Icinga-Refresh=1 in the response.

@nilmerg nilmerg added area/javascript Affects the javascript framework area/framework Affects third party integration/development labels Sep 6, 2023
@nilmerg nilmerg self-assigned this Sep 6, 2023
@cla-bot cla-bot bot added the cla/signed label Sep 6, 2023
@nilmerg nilmerg force-pushed the do-not-refresh-twice-upon-a-close-redirect branch from 7e3325b to 03f6288 Compare September 7, 2023 07:42
This allows the temporary adjustment of the autorefresh
interval.
@nilmerg nilmerg force-pushed the do-not-refresh-twice-upon-a-close-redirect branch from c6276d8 to 8f5b62f Compare September 7, 2023 10:07
@nilmerg nilmerg merged commit cdfe7e9 into master Sep 7, 2023
12 checks passed
@nilmerg nilmerg deleted the do-not-refresh-twice-upon-a-close-redirect branch September 7, 2023 10:50
@nilmerg nilmerg added this to the 2.12.0 milestone Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/framework Affects third party integration/development area/javascript Affects the javascript framework cla/signed
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant