-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
fix(user-preference-currency-display): remove unused prop ethLogoHeight #24517
Open
legobeat
wants to merge
1
commit into
MetaMask:develop
Choose a base branch
from
legobeat:remove-ethlogoheight
base: develop
Could not load branches
Branch not found: {{ refName }}
Could not load tags
Nothing to show
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+0
−10
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
legobeat
force-pushed
the
remove-ethlogoheight
branch
11 times, most recently
from
May 15, 2024 00:34
5b965d9
to
1dfa429
Compare
legobeat
force-pushed
the
remove-ethlogoheight
branch
3 times, most recently
from
May 21, 2024 09:38
71a425a
to
bc330bd
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #24517 +/- ##
===========================================
- Coverage 65.67% 65.67% -0.00%
===========================================
Files 1360 1360
Lines 54105 54105
Branches 14066 14066
===========================================
- Hits 35532 35531 -1
- Misses 18573 18574 +1 ☔ View full report in Codecov by Sentry. |
legobeat
force-pushed
the
remove-ethlogoheight
branch
4 times, most recently
from
May 21, 2024 11:57
c14fb33
to
0d0e652
Compare
legobeat
force-pushed
the
remove-ethlogoheight
branch
28 times, most recently
from
May 27, 2024 16:40
a1adb34
to
6b1c674
Compare
Fixes: Warning: React does not recognize the `ethLogoHeight` prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase `ethlogoheight` instead. If you accidentally passed it from a parent component, remove it from the DOM element. in div (created by ForwardRef) in ForwardRef (created by CurrencyDisplay) in CurrencyDisplay (created by UserPreferencedCurrencyDisplay) in UserPreferencedCurrencyDisplay (created by ConfirmTitle) in h3 (created by ForwardRef) in ForwardRef (created by ForwardRef) in ForwardRef (created by ConfirmTitle) in ConfirmTitle (created by ConfirmTransactionBase) in div (created by ConfirmPageContainerSummary) in div (created by ConfirmPageContainerSummary) in ConfirmPageContainerSummary (created by ConfirmPageContainerContent) in div (created by ConfirmPageContainerContent) in ConfirmPageContainerContent (created by ConfirmPageContainer) in div (created by ConfirmPageContainer) in GasFeeContextProvider (created by ConfirmPageContainer) in ConfirmPageContainer (created by ConfirmTransactionBase) in TransactionModalContextProvider (created by ConfirmTransactionBase) in ConfirmTransactionBase (created by ConnectFunction) in ConnectFunction (created by Context.Consumer) in withRouter(Connect(ConfirmTransactionBase)) (created by ConfirmSendEther) in ConfirmSendEther in LegacyMetaMetricsProvider (created by Wrapper) in LegacyI18nProvider (created by Wrapper) in I18nProvider (created by Wrapper) in Router (created by Wrapper) in Provider (created by Wrapper) in Wrapper 71 | const { history, Wrapper } = createProviderWrapper(store, pathname); 72 | return { > 73 | ...render(component, { wrapper: Wrapper }), | ^ 74 | history, 75 | }; 76 | }
legobeat
force-pushed
the
remove-ethlogoheight
branch
from
May 28, 2024 10:17
6b1c674
to
3940b21
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Fixes
console.error
:Related issues
AssetPickerAmount
#23192Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist