-
Notifications
You must be signed in to change notification settings - Fork 34
(FIX): Fixed the Founder Tab redirection error to Browse Tab on clcking on View Projects #391
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…ng on View Projects
What you did is wrong. You can fix it with just an |
Okaay fixing this and pushing a new commit @miladsoft !! |
Hey @miladsoft reverted back my changes in the NavMenu.razor file and applied the patch you suggested but it doesn't seem to work properly. I have attached a recording for both my code changes and live app. error.webm |
|
Yes @miladsoft thankssss !! Works properly. Pushing in the commit ! |
Pushed the new commit @miladsoft !! Screencast.from.2025-05-18.13-11-10.webm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well done!
@@ -889,6 +890,20 @@ | |||
StateHasChanged(); | |||
} | |||
} | |||
protected override void OnParametersSet() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
space between methods
@@ -890,6 +891,21 @@ | |||
} | |||
} | |||
|
|||
protected override void OnParametersSet() | |||
{ | |||
base.OnParametersSet(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we need this code also in OnParametersSet
?
Issue
When a project under the Founders Tab is opened it gets automatically redirected to the Browse Tab. This PR fixes this issue
The issue is demonstrated in this video below --
Screencast.from.2025-05-17.17-06-39.webm
Proof of Work
Working Video --
updated.webm
This video shows that it does not get redirected to the Browse Tab. On clicking on a Project under the Founders Tab it displays the same behavior as opening a project from the Browse Tab does.