Skip to content

#1247 Variable rename#1251

Merged
MoesSURF merged 6 commits into
mainfrom
variable-rename
Jul 2, 2026
Merged

#1247 Variable rename#1251
MoesSURF merged 6 commits into
mainfrom
variable-rename

Conversation

@ricardovdheijden

Copy link
Copy Markdown
Contributor

Comment on lines 94 to +131
@@ -129,9 +126,9 @@ public LoginController(UserRepository userRepository,
this.authenticationRequestRepository = authenticationRequestRepository;
this.securityContextRepository = securityContextRepository;
this.createFromInstitutionAllowedReturnDomains = createFromInstitutionProperties.getReturnUrlAllowedDomains();
this.spBaseUrl = spBaseUrl;
this.spServiceDeskBaseUrl = spServiceDeskBaseUrl;
this.myconextLoginUrl = myConextUrl + "/oauth2/authorization/my_conext";
this.myconextRedirectUrl = myconextRedirectUrl;
this.servicedeskRedirectUrl = servicedeskRedirectUrl;
this.myconextLoginUrl = myconextBasePath + "/oauth2/authorization/my_conext";

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI: this is the translation of the variables that end up in the frontend

@MoesSURF MoesSURF merged commit eefce5b into main Jul 2, 2026
5 checks passed
@github-project-automation github-project-automation Bot moved this from New to Delivered in Openconext-Myconext Jul 2, 2026
@MoesSURF MoesSURF deleted the variable-rename branch July 2, 2026 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Delivered

Development

Successfully merging this pull request may close these issues.

2 participants