Tychobra / polished

Authentication and Administration for Shiny apps
https://polished.tech
Other
234 stars 36 forks source link

added warning for firebase #161

Closed muschellij2 closed 2 years ago

muschellij2 commented 2 years ago

Fixing #160 with a warning (if Firebase is the only way to do these social sign ons)

merlinoa commented 2 years ago

Thanks John. Yes this is a good addition. Patrick moved the check to global_session_config() rather than in the Session R6 class. This way we will hit the warning as soon as we can detect it. I went ahead and merged Patrick's PR, so I'm going to go ahead and close this PR out.