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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
[v4][ContextualSaveBar] Remove withContext #1498
Conversation
601b402
to
9ba173e
Compare
withContext<Props, WithAppProviderProps, FrameContextType>( | ||
FrameContext.Consumer, | ||
), | ||
withAppProvider(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was never being used
9ba173e
to
57d5742
Compare
@dleroux This should hold up against the rendering issue but let's test tomorrow to be sure |
57d5742
to
b2dd51b
Compare
b2dd51b
to
c340b4d
Compare
alignContentFlush, | ||
}); | ||
}, | ||
// eslint-disable-next-line react-hooks/exhaustive-deps |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Frame context next changes so we don't need it as a dep
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
馃悜 馃嚠馃嚬
c340b4d
to
26f3ac2
Compare
WHY are these changes introduced?
Part of #1484
WHAT is this pull request doing?
I'm converting ContextualSaveBar to a functional component and removing
withContext
How to 馃帺
Tests / percy / development server