[pve-devel] [PATCH yew-mobile-gui 3/3] page login: skip consent banner when openid authorization is present

[Shannon Sterz <s.sterz@proxmox.com>]

improves ux so users don't have to agree to the consent banner twice.

Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>
---
 src/pages/page_login.rs | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/pages/page_login.rs b/src/pages/page_login.rs
index 7875af3..63fad30 100644
--- a/src/pages/page_login.rs
+++ b/src/pages/page_login.rs
@@ -10,6 +10,7 @@ use pwt::widget::{Button, Column, Row};
 use crate::widgets::TopNavBar;
 
 use proxmox_yew_comp::layout::card::standard_card;
+use proxmox_yew_comp::utils::openid_redirection_authorization;
 use proxmox_yew_comp::{LoginPanel, Markdown};
 
 use proxmox_login::Authentication;
@@ -68,7 +69,9 @@ impl Component for PvePageLogin {
             props.consent_text.as_ref().filter(|t| !t.is_empty())
         };
 
-        let content: Html = if let Some(consent_text) = consent_text {
+        let content: Html = if let Some(consent_text) = consent_text
+            && openid_redirection_authorization().is_none()
+        {
             let card = standard_card(tr!("Consent"), (), ())
                 .class("pwt-scheme-neutral")
                 .with_child(
-- 
2.47.3



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel