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

Add support for "kiosk" URL parameter #55

Merged
merged 1 commit into from
Feb 21, 2019
Merged

Add support for "kiosk" URL parameter #55

merged 1 commit into from
Feb 21, 2019

Conversation

satterly
Copy link
Member

@satterly satterly commented Feb 21, 2019

Go to ... http://local.alerta.io:8000/alerts?kiosk=1

Once logged in, the screen will go to "full screen" mode and the top menu will disappear.

@mustaqeem
Copy link

Screenshot 2021-10-28 at 4 45 13 PM

It is giving the above error.
@satterly

@sp1r
Copy link

sp1r commented Dec 21, 2021

"kiosk" url parameter is passed to API calls
API knows nothing about it (I guess it actually shouldn't) and returns error

I think we should remove this parameter from API call or maybe change the way kiosk mode is enabled?

@sp1r
Copy link

sp1r commented Dec 21, 2021

This patch works for me:

--- a/src/store/modules/alerts.store.ts
+++ b/src/store/modules/alerts.store.ts
@@ -160,6 +160,7 @@ const actions = {
         moment().utc().add(state.filter.dateRange[1], 'seconds').toISOString() // seconds offset
       )
     }
+    params.delete('kiosk')
 
     return AlertsApi.getAlerts(params)
       .then(({ alerts, total, pageSize }) => commit('SET_ALERTS', [alerts, total, pageSize]))
@@ -263,6 +264,7 @@ const actions = {
         moment().utc().add(state.filter.dateRange[1], 'seconds').toISOString() // seconds offset
       )
     }
+    params.delete('kiosk')
 
     return AlertsApi.getEnvironments(params)
       .then(({ environments }) => commit('SET_ENVIRONMENTS', environments))
-- 

@satterly satterly mentioned this pull request Jan 1, 2022
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.

3 participants