Skip to content

Commit 44c83be

Browse files
committed
fix(InstallWizard): Fix exception when InstallWizard completed
Since we don't have tests for the submit flow, we did not catch a regression from #13798 where the event handler was not bound to the class instance, causing an `this.setState` is not defined error when submitting the welcome form. This patch fixes the issue. Will add tests to cover this soon after.
1 parent 13649ec commit 44c83be

File tree

1 file changed

+5
-8
lines changed
  • src/sentry/static/sentry/app/views

1 file changed

+5
-8
lines changed

src/sentry/static/sentry/app/views/app.jsx

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -200,16 +200,13 @@ class App extends React.Component {
200200
e.stopPropagation();
201201
}
202202

203-
onConfigured() {
204-
this.setState({needsUpgrade: false});
203+
onConfigured = () => this.setState({needsUpgrade: false};
205204
}
206205

207-
handleNewsletterConsent = () => {
208-
// this is somewhat hackish
209-
this.setState({
210-
newsletterConsentPrompt: false,
211-
});
212-
};
206+
// this is somewhat hackish
207+
handleNewsletterConsent = () => this.setState({
208+
newsletterConsentPrompt: false,
209+
});
213210

214211
handleGlobalModalClose = () => {
215212
if (!this.mainContainerRef) {

0 commit comments

Comments
 (0)