From b6c659ccb85f2428e1e4b3b57e1bf259efc0eeb9 Mon Sep 17 00:00:00 2001
From: "Adolfo R. Brandes" <adolfo@axim.org>
Date: Sun, 29 Jun 2025 13:09:17 +0200
Subject: [PATCH] feat: redirect to route role on successful login

Now we have a way to discover app URLs dynamically; do so for the
(default) redirect on login.
---
 src/login/data/service.js | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/src/login/data/service.js b/src/login/data/service.js
index 572ab06f..086c8ed0 100644
--- a/src/login/data/service.js
+++ b/src/login/data/service.js
@@ -1,4 +1,4 @@
-import { getAuthenticatedHttpClient, getSiteConfig } from '@openedx/frontend-base';
+import { getAuthenticatedHttpClient, getSiteConfig, getUrlByRouteRole } from '@openedx/frontend-base';
 import * as QueryString from 'query-string';
 
 export async function loginRequest(creds) {
@@ -17,8 +17,11 @@ export async function loginRequest(creds) {
       throw (e);
     });
 
+  const defaultRedirectUrl = getUrlByRouteRole('org.openedx.frontend.role.dashboard');
+  const redirectUrl = data.redirect_url ?? defaultRedirectUrl;
+
   return {
-    redirectUrl: data.redirect_url || `${getSiteConfig().lmsBaseUrl}/dashboard`,
-    success: data.success || false,
+    redirectUrl,
+    success: data.success ?? false,
   };
 }