Skip to content

VSB-TUO/Removed forgot site to be unavailable - #1005

Merged
milanmajchrak merged 1 commit into
customer/vsb-tuofrom
vsb-tuo/hide-forgot-password-option
Sep 23, 2025
Merged

VSB-TUO/Removed forgot site to be unavailable#1005
milanmajchrak merged 1 commit into
customer/vsb-tuofrom
vsb-tuo/hide-forgot-password-option

Conversation

@jr-rk

@jr-rk jr-rk commented Sep 22, 2025

Copy link
Copy Markdown
Phases MP MM MB MR JM Total
ETA 0 0 0 0 0 0
Developing 0 0 0 0 0 0
Review 0 0 0 0 0 0
Total - - - - - 0
ETA est. 0
ETA cust. - - - - - 0

Problem description

Removed the unwanted option to register new users to VSB DSpace by removing the "forgot password" link from the login component's HTML template and completely eliminating the /forgot route from the application routing.

@jr-rk jr-rk self-assigned this Sep 22, 2025
@jr-rk
jr-rk requested a review from Copilot September 22, 2025 17:58

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes the "forgot password" functionality from the VSB DSpace application to prevent unwanted user registrations by eliminating the forgot password link from the login interface and removing the entire /forgot route.

  • Removed forgot password component imports and declarations from the theme module
  • Eliminated forgot password route configuration and related imports
  • Removed forgot password link from the login component HTML template

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
src/themes/custom/lazy-theme.module.ts Removed imports and declarations for forgot password components
src/app/shared/log-in/methods/password/log-in-password.component.ts Removed forgot password route import and getForgotRoute() method
src/app/shared/log-in/methods/password/log-in-password.component.html Removed forgot password link from login dropdown
src/app/app-routing.module.ts Removed forgot password route configuration and path import

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jr-rk
jr-rk requested a review from Copilot September 23, 2025 08:32

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jr-rk
jr-rk requested a review from milanmajchrak September 23, 2025 08:32
@milanmajchrak
milanmajchrak merged commit cc8b0c9 into customer/vsb-tuo Sep 23, 2025
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants