Skip to content

Conversation

minhqto
Copy link
Contributor

@minhqto minhqto commented Aug 21, 2025

Description

Resolves #217

This PR adds an active peer push notification that informs users if there are fellow bitchatters nearby. This is similar to iOS's push notification for active peers.

The active peer push notification will be sent if:

  • app is in the background
  • it's been more than 5 minutes since last notification
  • newly active peer was not already online previously

For the future, we should add the capability to customize push notifications. As bitchat gets more popular, this push notification might become "spammy".

4aaeeee5-1c22-49bd-8a28-2f2e296a11f1

Checklist

@minhqto minhqto marked this pull request as ready for review August 21, 2025 19:36
@minhqto
Copy link
Contributor Author

minhqto commented Aug 24, 2025

@callebtc any chance you can review this one? 🙏

@callebtc
Copy link
Collaborator

@callebtc any chance you can review this one? 🙏

thank you for this! I thought we had this already, might have gone missing in combat? could you remove the conflicts before I review and merge?

* main: (22 commits)
  Nip17 dms to extend the mesh  (permissionlesstech#313)
  bump to 1.2.0
  fix nip-17 (permissionlesstech#305)
  Geohash-specific relays (permissionlesstech#306)
  fix teleport tag and icons (permissionlesstech#303)
  UI changes and fixes (permissionlesstech#301)
  country -> region, region –> province (permissionlesstech#298)
  slightly increase padding (permissionlesstech#295)
  New about sheet (permissionlesstech#294)
  bump version (permissionlesstech#293)
  Statusbar seamless (permissionlesstech#289)
  Geohash notifications (permissionlesstech#288)
  Link preview (permissionlesstech#286)
  user bottom sheet (permissionlesstech#285)
  persist last channel (permissionlesstech#284)
  bump version code (permissionlesstech#283)
  UI fixes 2 (permissionlesstech#282)
  Scroll fix (permissionlesstech#281)
  fix minify errors (permissionlesstech#280)
  bump version (permissionlesstech#277)
  ...
@minhqto
Copy link
Contributor Author

minhqto commented Aug 25, 2025

@callebtc any chance you can review this one? 🙏

thank you for this! I thought we had this already, might have gone missing in combat? could you remove the conflicts before I review and merge?

@callebtc done! There are some strange formatting that my IDE is auto correcting that is caught up in this PR. Is that ok or do you prefer the previous formatting?

@callebtc callebtc added enhancement New feature or request needs review Someone should look at it UI/UX Improving interactions labels Aug 25, 2025
@callebtc
Copy link
Collaborator

@minhqto this is great, thank you. please contribute more!

@callebtc callebtc merged commit 02d5466 into permissionlesstech:main Aug 28, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs review Someone should look at it UI/UX Improving interactions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Notify when a user has been detected nearby.
2 participants