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

Reply feature does not work properly #39

Closed
0xf104a opened this issue Nov 21, 2022 · 5 comments
Closed

Reply feature does not work properly #39

0xf104a opened this issue Nov 21, 2022 · 5 comments

Comments

@0xf104a
Copy link
Owner

0xf104a commented Nov 21, 2022

    I would like to share my own experience concerning this issue.

I also am on GrapheneOS (latest) and latest NextcloudServices (v1.0-beta12)

Notifications are working on my device, including talk/spreed. However, the "Reply" functionality in the Android notification, doesn't send message. @TheDarkula do you still have this issue ?

Screenshot_20221121-113253

Originally posted by @NablaMetal in #38 (comment)

@0xf104a
Copy link
Owner Author

0xf104a commented Nov 21, 2022

@NablaMetal Hello once again. Unfortunately my Nextcloud instance is currently down and I wouldn't be able to reproduce your issue, so can you please try grabbing any exceptions while responding with quick replies. Though probably the problem is that new notification APIs in S already dropped support of that APIs that I have used in this app, but it is still worth trying grep'ing some exceptions

@NablaMetal
Copy link

I'm not seeing anything suspicious in nextcloud logs, with level 0. Did you meant greping error on the server or the client (I mean the "nextcloud services" app) ?

@0xf104a
Copy link
Owner Author

0xf104a commented Nov 22, 2022

Yes, I mean grepping exceptions from client app as I do not really think that Nextcloud would change API for sending messages in talk on server side

@NablaMetal
Copy link

Sorry I don't know how to do that on Android, there's no log related option in the app and I'm no dev.

freeflyk added a commit to freeflyk/NextcloudServices that referenced this issue Jan 9, 2023
Use provided link to the conversation instead of the object_id filed in the notification to extract the chatroom id.

background in the settings_activity.xml was removed to make the app in the main branch run.
0xf104a pushed a commit that referenced this issue Jan 12, 2023
fix to issue #39 Reply feature does not work properly
@0xf104a
Copy link
Owner Author

0xf104a commented Jan 14, 2023

Reply feature is fixed in 1.0-beta13 by @freeflyk

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

No branches or pull requests

2 participants