Skip to content
This repository has been archived by the owner. It is now read-only.

Remove string from tab on persistent partition #7741

Merged
merged 2 commits into from Mar 15, 2017
Merged

Conversation

@cezaraugusto
Copy link
Contributor

cezaraugusto commented Mar 15, 2017

Auditors: @bsclifton

Fix #7716

Issue was caused by persistent partitions being set as string instead of number

Test Plan:

  • Open new session tab
  • Click on a target="_blank" link
  • New tab should have only numbers and not partition- string on its tab
Auditors: @bsclifton

Fix #7716
@cezaraugusto cezaraugusto added this to the 0.14.0 milestone Mar 15, 2017
@cezaraugusto cezaraugusto self-assigned this Mar 15, 2017
Auditors: @cezaraugusto

Test Plan:
run `npm run unittest -- --grep="tabContent components"`
Copy link
Contributor Author

cezaraugusto left a comment

❤️ ❤️ ❤️ ❤️ ❤️ ❤️

Copy link
Contributor Author

cezaraugusto left a comment

❤️ ❤️ ❤️ ❤️ ❤️ ❤️

Copy link
Member

bsclifton left a comment

++++!

@bsclifton bsclifton merged commit ef92417 into brave:master Mar 15, 2017
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
Copy link
Collaborator

jonathansampson left a comment

++

@cezaraugusto cezaraugusto deleted the cezaraugusto:hotfix/7716 branch Jul 25, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.