-
Notifications
You must be signed in to change notification settings - Fork 36
Remove references to allowBlazor from code #1288
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
base: main
Are you sure you want to change the base?
Remove references to allowBlazor from code #1288
Conversation
|
Does this make the page for /Teams/List dead code, or is there another entry point to it? |
It is referred to in some partial cshtmls from past events (nipd2020/homePartial.cshtml, nipd2021/homepartial.cshtml). To not break those i was going to leave it. |
I don't think it is critically important to preserve the signup procedure for a five year old event, but why not change these to /Teams/Signup? I'm just trying to keep us from needing to perpetually maintain code that nobody will ever run. |
That is fair. I removed all references to /List and replaced it with signup |
|
Thanks! I'd like @morganbr to ring in on this since he did the original signup conversion and may know of a wrinkle we have forgotten, but I don't see any issues. I'm not signing off just so we remember that Morgan has to look. |
#1115
This PR