Skip to content

Fix error notification #51

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

Merged

Conversation

geichelberger
Copy link
Contributor

The error notification is shown if two or more streams are available, but it should only be displayed if there are no streams.

The error notification is shown if two or more streams are available, but
it should only be displayed if there are no streams.
@bluetom bluetom added bug Something which isn't working 4.5 labels Nov 10, 2024
@bluetom bluetom self-assigned this Nov 10, 2024
@bluetom bluetom added 4.5.1 and removed 4.5 labels Nov 12, 2024
@bluetom bluetom requested a review from ferishili November 22, 2024 08:35
Copy link
Contributor

@ferishili ferishili left a comment

Choose a reason for hiding this comment

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

I have first rebased this PR and added some minor improvements. Hence, I would say, it is good to go!

@geichelberger
Copy link
Contributor Author

Did you rebase it or merge a branch into that one because of a merge commit? What's the improvement? Because the error notification is now removed.

@ferishili
Copy link
Contributor

ferishili commented Nov 25, 2024

Did you rebase it or merge a branch into that one because of a merge commit? What's the improvement? Because the error notification is now removed.

Yes, I wrote that I rebase and merged. The part to handle that, was merged before via: f48a850

@geichelberger
Copy link
Contributor Author

So basically, a duplicate of that PR got merged before this one, which was created almost 3 months later.

@ferishili
Copy link
Contributor

Unfortunately, it does seem that way. It looks like this PR might have been impacted by changes in personnel and limited time for repository management when it was originally created. Apologies for any inconvenience, and thank you for your understanding.

@bluetom bluetom merged commit 89dbe1a into Opencast-Moodle:master Dec 1, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something which isn't working
Projects
Status: CLOSED
Development

Successfully merging this pull request may close these issues.

3 participants