In instances where _bptIn is empty, exit

[danielmkm]

Description

Please include a summary of the change and if relevant which issue is fixed. Please also include relevant motivation and context.

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Dependency changes
• Code refactor / cleanup
• Documentation or wording changes
• Other

How should this be tested?

Please provide instructions so we can test. Please also list any relevant details for your test configuration.

• Test A
• Test B

Visual context

Please provide any relevant visual context for UI changes or additions. This could be static screenshots or a loom screencast.

Checklist:

• I have performed a self-review of my own code
• I have requested at least 1 review (If the PR is significant enough, use best judgement here)
• I have commented my code where relevant, particularly in hard-to-understand areas
• If package-lock.json has changes, it was intentional.
• The base of this PR is master if hotfix, develop if not

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
beta-app-v2	✅ Ready (Inspect)	Visit Preview	Jul 20, 2023 4:16am

1 Ignored Deployment

Name	Status	Preview	Updated (UTC)
app-v2	⬜️ Ignored (Inspect)		Jul 20, 2023 4:16am

[timjrobinson]

It doesn't seem like it's possible for _bptIn to be an empty string if hasBptIn is false, as they are checking the same underlying value when isSingleAssetExit is false. Gareth has implemented another fix for this issue here: #3752 so closing in favor of that.