Skip to content
This repository has been archived by the owner on Jan 14, 2025. It is now read-only.

onNotification fires every time when the app goes from background to foreground #1455

Closed
landorid opened this issue May 23, 2020 · 2 comments
Closed

onNotification fires every time when the app goes from background to foreground #1455

landorid opened this issue May 23, 2020 · 2 comments
Assignees
@Dallas62

Comments

@landorid
Copy link

landorid commented May 23, 2020 •
edited
Loading

Hello!

During my work, I found a bug in my application. So I have tested your example application int this repo, where I was able to reproduce it:

  1. Build the application for Android.
  2. Click Local notification (now) button
  3. Kill the application
  4. Open it from the notification and close the Alert window
  5. Send the application to background
  6. Open it again

The Alert will pop up again, however, it displayed once before.

Expected behaviour:
Don't show the notification again, just once.

Thank you for your help!
@Dallas62 Dallas62 self-assigned this May 23, 2020
@Dallas62 Dallas62 mentioned this issue May 25, 2020
Closed
@landorid
Copy link
Author

I've tested and it works! Thank you for your fast fix!

vaibhavhrt added a commit to fcbtech/react-native-push-notification that referenced this issue Jun 24, 2020
@vaibhavhrt @andrewtremblay-pear
@Dallas62 @willnaoosmith @GabeConsalter @taymer
Sync with upstream (#3)
19b00e7 
* Make sure to import PushNotificationIOS from react-native first

* typo

* Delete versions.yml

* Update push-notification-ios to 1.2.0

Version 1.1.0 giving a `No known class method for selector 'didReceiveNotificationResponse'`error

* Add a Log.e() on fail.

* Add option in AndroidManifest to allow/remove notification in foreground (remote-only)

* Change the documentation of onNotification.

* Implement `onNewToken` for Firebase.

* Add an example with sound in project example.

* feat: ✨ Revoke Token method

Added the Revoke Token method to Android only. With this method is possible to generate a new token after a logout case with revoking.

* docs: 📝 Revoke Token method documentation

Added Revoke Token method descprition at Android Only Methods section

* chore: Added Revoke Token button

Updated example with the new Revoke Token method, adding a button, alert feedback and necessary calls

* Allow numbers for `number` and `id` in Android.

* chore: 👌 Revoke Token method renamed to Abandon Permissions

* chore: 👌 Updated to new Abandon Permissions method name

The Revoke Token method has been renamed to Abandon Permissions method

* docs: 📝 Updated with new Abandon Permissions method

Renamed Revoked Method to Abandon Permissions method. The method from iOS only section was removed and added a new general method section with the new method

* Changes in example Application.

* Use directly the callNative for `abandonPermissions`.

* Put deleteInstanceId() in a separate thread.

* Bump to version 3.4.0.

* Prevent crash for NaN.

* Update CHANGELOG.md.

* Prevent sound to be mute in default state.

* Missing change on variable name.

* feat: multiple push providers

fix: fix naming

fix: more fixes

fix: more fixes

fix: add activity service

fix: update naming

* Change from 18 months to 1 year

* feat: explicitly override onNewToken

* zo0r#1431

* Move back onNewToken to Firebase service.

* Bump to 3.5.0.

* Switch from support library to androidx.
zo0r#1449

* Bump to 3.5.1.

* Update README.md

* Fix sounds volume and DND is ignored.
Fix zo0r#1455 `onNotification fires every time when the app goes from background to foreground`

* Remove extra interface.

* Bump to 3.5.2.

* Fix changelog, wrong issue mentioned.

Co-authored-by: Hans Govind <hans@exove.fi>
Co-authored-by: Andrew Tremblay (Pear profile) <43054023+andrewtremblay-pear@users.noreply.github.com>
Co-authored-by: Boris Tacyniak <boris.tacyniak@free.fr>
Co-authored-by: willnaoosmit <55159911+willnaoosmit@users.noreply.github.com>
Co-authored-by: Gabriel Consalter <consaltergabriel@gmail.com>
Co-authored-by: sarah denaci <sarah@gridwise.io>
Co-authored-by: Taymer Ragazzini <tragazzini@gmail.com>
@strawberry-code
Copy link

Happening again.

"@react-native-community/push-notification-ios": "^1.0.6",
"react-native": "0.61.5",

buildToolsVersion = "29.0.2"
minSdkVersion = 21
compileSdkVersion = 29
targetSdkVersion = 29

minhnc added a commit to minhnc/react-native-push-notification that referenced this issue Mar 24, 2021
@minhnc
Merge commit '1d497e60a142b4a846907ec19b80ab72a5ae5c8d'
5ba16d8 
* commit '1d497e60a142b4a846907ec19b80ab72a5ae5c8d': (109 commits)
  Bump to 3.5.2.
  Remove extra interface.
  Fix sounds volume and DND is ignored. Fix zo0r#1455 `onNotification fires every time when the app goes from background to foreground`
  Update README.md
  Bump to 3.5.1.
  Switch from support library to androidx. zo0r#1449
  Bump to 3.5.0.
  Move back onNewToken to Firebase service.
  zo0r#1431
  feat: explicitly override onNewToken
  Change from 18 months to 1 year
  feat: multiple push providers
  Missing change on variable name.
  Prevent sound to be mute in default state.
  Update CHANGELOG.md.
  Prevent crash for NaN.
  Bump to version 3.4.0.
  Put deleteInstanceId() in a separate thread.
  Use directly the callNative for `abandonPermissions`.
  Changes in example Application.
  ...

# Conflicts:
#	android/src/main/java/com/dieam/reactnativepushnotification/modules/RNPushNotificationListenerService.java
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@Dallas62 Dallas62

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@landorid @Dallas62 @strawberry-code