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

Unifiedpush with NextPush #409

Open
1 task done
timechips opened this issue Dec 8, 2024 · 5 comments
Open
1 task done

Unifiedpush with NextPush #409

timechips opened this issue Dec 8, 2024 · 5 comments
Labels
feature-request New feature or request

Comments

@timechips
Copy link

Is there an existing request for this?

  • I have searched the existing issues

Feature description

The unifiedpush worked with NextPush on the seperate app, but since the apps merged I cannot seem to figure out how to set it up. The only option molly gives me is to scan the QR code for molly socket.

@timechips timechips added the feature-request New feature or request label Dec 8, 2024
@phoropter
Copy link
Contributor

phoropter commented Dec 9, 2024

UnifiedPush requires mollysocket. That was always the case. What most likely happened before is that your app reverted to websocket for notifications.

If you cannot/ don't want to host your own mollysocket, then there are services that host it publicly, like https://molly.adminforge.de/

@timechips
Copy link
Author

timechips commented Dec 9, 2024

UnifiedPush requires mollysocket. That was always the case. What most likely happened before is that your app reverted to websocket for notifications.

If you cannot/ don't want to host your own mollysocket, then there are services that host it publicly, like https://molly.adminforge.de/

Reinstalled the depricated molly unifiedpush app just to check. I logged in and connected to NextPush. I'm pretty sure it is using unified push from my Nextcloud server. Correct me if I'm wrong.

Screenshot_20241209-195230.png

Screenshot_20241209-195235.png

Screenshot_20241209-195256.png

@phoropter
Copy link
Contributor

You can see in the 1st & 2nd screenshots that it's giving you warnings about no mollysocket server. I'm pretty sure the app will swap to websocket after enough time without a mollysocket connection and you won't get timely notifications until it does so.

@timechips
Copy link
Author

I had this config for over half a year. Worked flawlessly, only time it did not was when my server was down, at that time NextPush notified me so (I did not get notifications when the server was down). All the notifications were on time (almost always under 5sec, and never over 30). No exessive battery usage. Never seen "WebSocket Service" notification. All the people using NextPush also had molly-socket configured, I did not. Was this just websocket? And if it was, why did I not get notifications when my server was down?

@p1gp1g
Copy link
Member

p1gp1g commented Dec 29, 2024

The previous UX could be confusing and let some user think they were using UP while Molly was using the websocket (which we improved a bit). For instance, on your screenshot you can see the warning next to "UnifiedPush" and "MollySocket server not found" for the status.

We made the QR code mandatory for many reasons, one of them is to ensure the user has a mollysocket server running. Sorry for the confusion

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request New feature or request
Development

No branches or pull requests

3 participants