Skip to content

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

Merged
mlykotom merged 24 commits into
android:mainfrom
shangeethsivan:main
Jan 10, 2025
Merged

[JetChat] Add glance widget for JetChat App #1424#1425
mlykotom merged 24 commits into
android:mainfrom
shangeethsivan:main

Conversation

@shangeethsivan

@shangeethsivan shangeethsivan commented Jul 3, 2024

Copy link
Copy Markdown
Contributor

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
@shangeethsivan
shangeethsivan requested a review from arriolac 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

@mlykotom mlykotom left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for your PR!

Comment thread Jetchat/app/src/main/AndroidManifest.xml Outdated
Comment thread Jetchat/app/src/main/res/values/strings.xml Outdated
Comment thread Jetchat/app/src/main/res/values/strings.xml Outdated
Comment thread Jetchat/app/src/main/res/layout/widget_loading_layout.xml Outdated
Comment thread Jetchat/app/src/main/java/com/example/compose/jetchat/components/JetchatDrawer.kt Outdated
Comment thread Jetchat/app/src/main/java/com/example/compose/jetchat/components/JetchatDrawer.kt Outdated
Comment thread Jetchat/app/src/main/java/com/example/compose/jetchat/data/FakeData.kt Outdated
Comment thread Jetchat/app/src/main/res/layout/widget_error_layout.xml Outdated
Comment thread Jetchat/app/src/main/res/layout/widget_error_layout.xml Outdated
@mlykotom
mlykotom requested a review from secondsun October 21, 2024 12:08

@secondsun secondsun left a comment

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Contributor Author

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

@shangeethsivan

Copy link
Copy Markdown
Contributor Author

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

@secondsun secondsun left a comment

Copy link
Copy Markdown
Contributor

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.

Comment thread Jetchat/app/src/main/java/com/example/compose/jetchat/components/JetchatDrawer.kt Outdated
Comment thread Jetchat/app/src/main/java/com/example/compose/jetchat/components/JetchatDrawer.kt Outdated

@mlykotom mlykotom left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pls check some of the comments, after that it's lgtm :)

Thank you and sorry for the long turnaround!

Comment thread Jetchat/app/src/main/java/com/example/compose/jetchat/components/JetchatDrawer.kt Outdated
Comment thread Jetchat/app/src/main/java/com/example/compose/jetchat/JetchatApplication.kt Outdated
…kt class as initially implemented. Also propagating the requiresApi check to WidgetDiscoverability composable.
@shangeethsivan

shangeethsivan commented Jan 9, 2025

Copy link
Copy Markdown
Contributor Author

@mlykotom I have addressed the last set of comments hope this is the last one 😅 . Also the issue that was related to this PR has auto closed should I reopen it ?

#1424 - Im not seeing a reopen option here.

@mlykotom

mlykotom commented Jan 9, 2025

Copy link
Copy Markdown
Contributor

There's one more thing -- the unused MessagesRepository + pls fix the format. Otherwise this is LGTM :)

@mlykotom
mlykotom merged commit 8abb7d2 into android:main Jan 10, 2025
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

4 participants