fix: Handle book navigation failure more gracefully (BL-15692)

[andrew-polk]

Show back button and better message

---

This change is 

[hatton]

@hatton made 1 comment.
Reviewable status: 0 of 2 files reviewed, 1 unresolved discussion (waiting on @andrew-polk).

---

src/bloom-player-core.tsx line 934 at r1 (raw file):

            msg = "<p>We could not find that book.</p>";
        }

At this point, would it be correct to wrap the following in an else? I can't tell whether we think that we could get this message but then later decide to change the message. If we know that we would not change the message, then an else would make that clear.

[andrew-polk]

@andrew-polk made 1 comment.
Reviewable status: 0 of 2 files reviewed, 1 unresolved discussion (waiting on @hatton).

---

src/bloom-player-core.tsx line 934 at r1 (raw file):

Previously, hatton (John Hatton) wrote…

At this point, would it be correct to wrap the following in an else? I can't tell whether we think that we could get this message but then later decide to change the message. If we know that we would not change the message, then an else would make that clear.

Good catch. Fixed.

[github-actions]

🎉 This PR is included in version 2.18.0-alpha.3 🎉

The release is available on:

• npm package (@alpha dist-tag)
• GitHub release

Your semantic-release bot 📦🚀