-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
React-native: Fix event listening for story navigation #8690
Conversation
…ry rendered is called when switching tabs.
This pull request is being automatically deployed with ZEIT Now (learn more). 🔍 Inspect: https://zeit.co/storybook/monorepo/lnafdgqg7 |
@@ -50,6 +50,8 @@ export default class ActionLogger extends Component<ActionLoggerProps, ActionLog | |||
}; | |||
|
|||
addAction = (action: ActionDisplay) => { | |||
console.warn('called!!'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
???
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoops. :D
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Issue: #6471
What I did
Addons-ondevice-actions was completely broken (never showing any actions), so I changed the event to proper one. The issue #6471 still persists, but at least we can avoid it by not passing react events to it directy by doing.