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

Bug: Header back button not working in ignite-bowser 5.1.0 #347

Closed
patevs opened this issue May 25, 2020 · 3 comments · Fixed by #350
Closed

Bug: Header back button not working in ignite-bowser 5.1.0 #347

patevs opened this issue May 25, 2020 · 3 comments · Fixed by #350
Labels

Comments

@patevs
Copy link

patevs commented May 25, 2020

What's going on?

Back button in the Header component does not navigate to previous screen in ignite-bowser version 5.1.0

The hardware back button works as expected but the back button in the Header component does not.

I have only tested this on android so unsure if the issue is present on IOS.


Steps to reproduce

  1. Generate a new app with ignite-cli using ignite-bowser boilerplate.
  2. Run the generated app.
  3. Navigate from welcome-screen to demo-screen.
  4. Press the back button in the Header component; nothing happens.

@dakkafex
Copy link
Contributor

Could this be an issue with react-navigation?

@dakkafex
Copy link
Contributor

I figured out why...

in app/screens/demo-screen/demo-screen.tsx on line 82 for some reason there are 2 arrow functions starting

const goBack = () => () => navigation.goBack()

Remove one () => and it works fine again.

It's wierd that this doesnt throw an error or is even caught up my linter

@infinitered-circleci
Copy link

🎉 This issue has been resolved in version 5.1.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants