-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Remove direct usage for window.location in SideNavigationBarDirective.ts #6738
Comments
Hi @ankita240796 can i be assigned to this thanks |
Is this issue still open? If so, can I be assigned to it |
Hi @lkcbharath, I think you haven't signed the CLA yet, would you mind following the getting started wiki page first? |
Does this fix this issue? |
@xlogix, Thanks for the updated link, I've edited the issue thread and replaced the 404 link with a correct permalink. Re the fix: I would suggest you create a PR and let's discuss it over there! :) |
I would like to work on this issue. Please assign it to me. |
Direct usage of window.location should be changed in the file SideNavigationBarDirective.ts here.
Refer the comment here
The text was updated successfully, but these errors were encountered: