-
Notifications
You must be signed in to change notification settings - Fork 45
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
1413 vis feedback #1428
1413 vis feedback #1428
Conversation
…lement on all pinned courses
Merge error fixed
Your Testserver will be ready at https://1428.test.live.mm.rbg.tum.de in a few minutes. Logins
|
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.
The code looks good to me, but I think you might have implemented it for the wrong delete button (see review comments)
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.
👍
Motivation and Context
#1413
Description
The page refreshes after lecture deletion, giving visual feedback of changes
Steps for Testing
Prerequisites: