Skip to content
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

Fix selectors for Nextcloud Talk recipe #470

Conversation

eandersons
Copy link
Contributor

@eandersons eandersons commented Nov 26, 2023

Pre-flight Checklist

Please ensure you've completed all of the following.

Description of Change

Script and CSS was updated to add selectors for the latest Nextcloud (27).
Previous selectors were left as is for backwards compatibility.

Additionally direct notifications count now is detected by unread conversations count in the left sidebar, using Talk specific notifications as a fallback.

Fixes ferdium/ferdium-app#971

Script and CSS was updated to add selectors for the latest Nextcloud
(27).
Previous selectors were left as is for backwards compatibility.

Additionally direct notifications count now is detected by unread
conversations count in the left sidebar, using Talk specific
notifications as a fallback.

Fixes ferdium-app/#971
@mcmxcdev
Copy link
Member

This looks great codewise, but I can't verify this as I am not using Nextcloud Task. Deferring to other contributors.

@eandersons eandersons changed the title Fix selectors for Nextcloud Task recipe Fix selectors for Nextcloud Talk recipe Nov 27, 2023
@eandersons eandersons closed this Nov 27, 2023
@eandersons eandersons deleted the Fix-selectors-in-Nextcloud-Talk-recipe branch November 27, 2023 08:49
@eandersons
Copy link
Contributor Author

eandersons commented Nov 27, 2023

Sorry, I accidentally deleted the wrong branch.

I recreated the branch Fix-selectors-in-Nextcloud-Talk-recipe. Is it possible to reopen this pull request or should I create a new one?

@mcmxcdev
Copy link
Member

Oh too bad! Unfortunately, the GitHub interace tells me that I can't reopen the PR because it was recreated.

@mydarkstar
Copy link
Member

Duplicate of #473

@mydarkstar mydarkstar marked this as a duplicate of #473 Nov 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: No Notifications in Nextcloud Talk?
3 participants