marineenergy / apps

shiny apps for MHK-env
https://marineenergy.github.io/apps/
MIT License
3 stars 1 forks source link

report-v2: tables hanging / Bad Gateway; rm `observe(poll_rpts_tbl())`; strip out shinydashboardPlus to get mobile layout to work #66

Closed bbest closed 2 years ago

bbest commented 2 years ago

The R package RinteRface/shinydashboardPlus seems to be the culprit in not showing tables across the Management | Documents | Publications | Spatial tabs:

shiny.marineenergy.app/report-v2 image

And it (or combined with funky tab customization) also prevents mobile layout from showing content:

image

bbest commented 2 years ago

Ok, the inability for the table to show seems related to only when a user logs in, eg when trying on Safari not already logged in:

image

bbest commented 2 years ago

And here's the JS Console when logged in showing errors related to null properties readingstyle with onSignIn:

image

Uncaught TypeError: Cannot read properties of null (reading 'style')
    at onSignIn (shiny.marineenergy.app/:182)
    at k2.<anonymous> (apis.google.com/_/scs/apps-static/_/js/k=oz.gapi.en.ILxaxn5iF9E.O/m=signin2/rt=j/sv=1/d=1/ed=1/am=AQ/rs=AGLTcCP4GH4pdEaIiZe1k-cWfBRo9KTiAQ/cb=gapi.loaded_0:501)
    at apis.google.com/_/scs/apps-static/_/js/k=oz.gapi.en.ILxaxn5iF9E.O/m=signin2/rt=j/sv=1/d=1/ed=1/am=AQ/rs=AGLTcCP4GH4pdEaIiZe1k-cWfBRo9KTiAQ/cb=gapi.loaded_0:357
    at e.nr (apis.google.com/_/scs/apps-static/_/js/k=oz.gapi.en.ILxaxn5iF9E.O/m=signin2/rt=j/sv=1/d=1/ed=1/am=AQ/rs=AGLTcCP4GH4pdEaIiZe1k-cWfBRo9KTiAQ/cb=gapi.loaded_0:234)
    at hk (apis.google.com/_/scs/apps-static/_/js/k=oz.gapi.en.ILxaxn5iF9E.O/m=signin2/rt=j/sv=1/d=1/ed=1/am=AQ/rs=AGLTcCP4GH4pdEaIiZe1k-cWfBRo9KTiAQ/cb=gapi.loaded_0:237)
    at ck (apis.google.com/_/scs/apps-static/_/js/k=oz.gapi.en.ILxaxn5iF9E.O/m=signin2/rt=j/sv=1/d=1/ed=1/am=AQ/rs=AGLTcCP4GH4pdEaIiZe1k-cWfBRo9KTiAQ/cb=gapi.loaded_0:237)
    at _.Nj.Wt (apis.google.com/_/scs/apps-static/_/js/k=oz.gapi.en.ILxaxn5iF9E.O/m=signin2/rt=j/sv=1/d=1/ed=1/am=AQ/rs=AGLTcCP4GH4pdEaIiZe1k-cWfBRo9KTiAQ/cb=gapi.loaded_0:236)
    at Lj (apis.google.com/_/scs/apps-static/_/js/k=oz.gapi.en.ILxaxn5iF9E.O/m=signin2/rt=j/sv=1/d=1/ed=1/am=AQ/rs=AGLTcCP4GH4pdEaIiZe1k-cWfBRo9KTiAQ/cb=gapi.loaded_0:229)
bbest commented 2 years ago

The hanging issue was fixed by removing the observe(poll_rpts_tbl()) per commit https://github.com/marineenergy/apps/commit/46ab2f802253d8ded6e3e73fad068cb28d782683.