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

[JetChat] Add glance widget for JetChat App #1424 #1425

Open
wants to merge 19 commits into
base: main
Choose a base branch
from

Conversation

shangeethsivan
Copy link

@shangeethsivan shangeethsivan commented Jul 3, 2024

Added an Unreads widget for the JetChat app using Glance. Also has a option to add widget to home screen on the app.

Glance.Widget.Recording.webm

Fixes #1424

@shangeethsivan shangeethsivan requested a review from a team as a code owner July 3, 2024 16:34
@riggaroo riggaroo removed the request for review from arriolac August 14, 2024 14:21
@shangeethsivan shangeethsivan changed the base branch from main to tm/root-project October 17, 2024 06:23
@shangeethsivan shangeethsivan changed the base branch from tm/root-project to main October 17, 2024 06:23
Copy link
Contributor

@mlykotom mlykotom left a comment

Choose a reason for hiding this comment

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

Thanks for your PR!

Copy link

@secondsun secondsun left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution! I've added my feedback, and +1 all of @mlykotom's points.

Are you expecting to add live data to the widget too, or only stick with static data?

@shangeethsivan
Copy link
Author

Thanks for the review @secondsun and @mlykotom will work on these over the weekend and will re-request a review.

@shangeethsivan
Copy link
Author

@secondsun @mlykotom Can you guys take a look at the PR now ?

Copy link

@secondsun secondsun left a comment

Choose a reason for hiding this comment

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

Found one thing with linting. Line is too long.

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.

[FR]: Glance Widget for JetChat App
3 participants