Skip to content

feat: expect NSString token instead NSData #1

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.

Sign up for GitHub

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

Open
wants to merge 19 commits into
base: master
Choose a base branch
from
Open

feat: expect NSString token instead NSData #1

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
dffaa85
fix(ios): edge case when non-interactive notification is tapped
mvayngrib Apr 3, 2019
ec2d6d7
fix: mis-emission of notification opened event
mvayngrib Apr 3, 2019
8066938
fix: propagate error when user denies permission
mvayngrib Apr 4, 2019
3edca5f
rename to publish under @exodus in npm
mvayngrib Apr 4, 2019
b29368e
1.3.1
mvayngrib Apr 4, 2019
1e9dfc4
fix: specify requiresMainQueueSetup -> YES, to remove annoying warning
mvayngrib Apr 4, 2019
83ea4c6
1.3.2
mvayngrib Apr 4, 2019
f5a3011
feat: support requesting local-only notifications
mvayngrib Apr 5, 2019
6b53f9d
1.4.0
mvayngrib Apr 5, 2019
c959e8a
fix: default requestPermissions to remote:true
mvayngrib Apr 5, 2019
96fdfba
1.4.1
mvayngrib Apr 5, 2019
1d144e8
fix: use RCTConvert to interpret options
mvayngrib Apr 5, 2019
c35a2e8
1.4.2
mvayngrib Apr 5, 2019
c11cd6b
feat: expect NSString token instead NSData
diegomura Jun 14, 2021
0146cae
fix: add tabs back to RNNRouter.m
diegomura Jun 14, 2021
66d2fd9
fix: add tabs back to RNNRouter.m
diegomura Jun 14, 2021
bf29ea8
fix: add tabs back to RNNotifications.m
diegomura Jun 14, 2021
b93646c
fix: add tabs back to RNNotifications.m
diegomura Jun 14, 2021
1201a22
fix: add tabs back to AppDelegate.m
diegomura Jun 14, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: specify requiresMainQueueSetup -> YES, to remove annoying warning
  • Loading branch information
@mvayngrib
mvayngrib committed Apr 4, 2019
commit 1e9dfc438ca6d9897ac283a41c43ba081d2840c4
6 changes: 5 additions & 1 deletion RNNotifications/RNNotifications.m
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -174,7 +174,11 @@ - (instancetype)init
return self;
}


+ (BOOL)requiresMainQueueSetup
{
// TODO: figure out if we can return NO
return YES;
}

- (void)setBridge:(RCTBridge *)bridge
{
Expand Down