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.

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

Jump to bottom

'No protocol method IDeref...' when opening notifications settings #19012

Closed
pavloburykh opened this issue Feb 27, 2024 · 1 comment · Fixed by #19014
Closed

'No protocol method IDeref...' when opening notifications settings #19012

pavloburykh opened this issue Feb 27, 2024 · 1 comment · Fixed by #19014
Assignees
@siddarthkay
Labels
e2e test blocker
Milestone
2.28.0 Alpha

Comments

@pavloburykh
Copy link
Contributor

Steps:

  1. Go to Profile => Notifications
  2. Observe the result

Actual result: 'No protocol method IDeref. -deref defined for type underfined' when opening Notifications settings

photo_2024-02-27 12 05 32
Status-debug-logs - 2024-02-27T120507.781.zip

telegram-cloud-document-2-5400192398510799787.mp4

Additional Information

  • Status version: nightly
  • Operating System: Android, iOS
@pavloburykh pavloburykh added this to the 2.28.0 Alpha milestone Feb 27, 2024
@siddarthkay siddarthkay self-assigned this Feb 27, 2024
@flexsurfer
Copy link
Member

ERROR re-frame: no subscription handler registered for: :push-notifications/wallet-transactions. Returning a nil subscription.
ERROR re-frame: no subscription handler registered for: :push-notifications/wallet-transactions. Returning a nil subscription.
most likely was removed here #18749

@siddarthkay siddarthkay mentioned this issue Feb 27, 2024
Merged
siddarthkay added a commit that referenced this issue Feb 27, 2024
@siddarthkay
fix: cleanup wallet push notification subscription
3688e1e 
fixes #19012

### Summary

The legacy wallet code was removed here -> #18749
as part of that purge we also cleaned up re-frame subscriptions for wallet push notifications.

Since the new wallet is under active development we would not need this toggle in settings just yet.

This commit fixes the app crashing on settings UI.

## Platforms
- Android
- iOS
siddarthkay added a commit that referenced this issue Feb 27, 2024
@siddarthkay
fix: cleanup wallet push notification subscription (#19014)
bf1a734 
fixes #19012

### Summary

The legacy wallet code was removed here -> #18749
as part of that purge we also cleaned up re-frame subscriptions for wallet push notifications.

Since the new wallet is under active development we would not need this toggle in settings just yet.

This commit fixes the app crashing on settings UI.

## Platforms
- Android
- iOS
@ilmotta ilmotta removed the bug label Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@siddarthkay siddarthkay

Labels
e2e test blocker
Projects
None yet
Milestone
2.28.0 Alpha
Development

Successfully merging a pull request may close this issue.

fix: cleanup wallet push notification subscription status-im/status-mobile
4 participants
@ilmotta @flexsurfer @siddarthkay @pavloburykh