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

Correct the method for detecting unread messages for Google Chat #549

Closed
wants to merge 4 commits into from
Closed

Correct the method for detecting unread messages for Google Chat #549

wants to merge 4 commits into from

Conversation

Shampra
Copy link
Contributor

@Shampra Shampra commented May 15, 2024

Pre-flight Checklist

Please ensure you've completed all of the following.

Description of Change

Change the method for detecting unread messages, because using div classes is broken every time Google chat is updated (this is currently the case)

Copy link
Contributor

@vraravam vraravam left a comment

Choose a reason for hiding this comment

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

@Shampra - where are the actual changes?

Use now the ico link because using div classes is broken with every Google chat update
@Shampra
Copy link
Contributor Author

Shampra commented May 15, 2024

@vraravam My bad, wrong branch... fixed!

@SpecialAro
Copy link
Member

Hello @Shampra, thank you for logging this PR.

Can you please run pnpm validate and push the changes so builds don't fail?

@Shampra
Copy link
Contributor Author

Shampra commented May 15, 2024

Hmm, I don't understand why it doesn't like it. pnpm validate seems ok and change nothing.
I close, retry in my side and repost when functional and clean

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.

3 participants