[web] fix uncaught exception bug
ClosedPublic
Actions

Authored by varun on Tue, Jun 18, 8:19 PM.

Tags

None

Referenced Files

	F2129355: D12471.diff
	Thu, Jun 27, 1:00 PM

	Unknown Object (File)
	Wed, Jun 26, 11:46 PM

	Unknown Object (File)
	Wed, Jun 26, 1:17 PM

	Unknown Object (File)
	Wed, Jun 26, 12:47 PM

	Unknown Object (File)
	Wed, Jun 26, 11:20 AM

	Unknown Object (File)
	Tue, Jun 25, 3:28 PM

	Unknown Object (File)
	Tue, Jun 25, 11:08 AM

	Unknown Object (File)
	Mon, Jun 24, 9:13 PM

View All 17 Files

Subscribers

tomek

Details

Reviewers

ashoat
inka

Commits

rCOMMfd8be14a14af: [web] fix uncaught exception bug

Summary

we don't need to re-throw here. i think this was introduced in D11765, when we introduced new login hooks.

Test Plan

no longer getting an uncaught exception error when i enter an incorrect password

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

varun created this revision.Tue, Jun 18, 8:19 PM

Herald added a subscriber: tomek. · View Herald TranscriptTue, Jun 18, 8:19 PM

Harbormaster completed remote builds in B29766: Diff 41476.Tue, Jun 18, 8:36 PM

varun requested review of this revision.Tue, Jun 18, 8:36 PM

ashoat accepted this revision.Wed, Jun 19, 5:25 AM

This revision is now accepted and ready to land.Wed, Jun 19, 5:25 AM

Closed by commit rCOMMfd8be14a14af: [web] fix uncaught exception bug (authored by varun). · Explain WhyWed, Jun 19, 6:39 AM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMMfd8be14a14af: [web] fix uncaught exception bug.

Revision Contents
Changeset List

Path

Size

web/

account/

traditional-login-form.react.js

1 line

Diff 41489

View Options

web/account/traditional-login-form.react.js

[web] fix uncaught exception bugClosedPublicActions