[lib] Fix crash when logged out with sidebar open
ClosedPublic
Actions

Authored by ashoat on May 4 2023, 9:59 AM.

Tags

None

Referenced Files

	F2113703: D7719.id26084.diff
	Wed, Jun 26, 4:10 AM

	F2113640: D7719.id26081.diff
	Wed, Jun 26, 3:58 AM

	Unknown Object (File)
	Mon, Jun 24, 12:58 PM

	Unknown Object (File)
	Mon, Jun 24, 5:51 AM

	Unknown Object (File)
	Sun, Jun 23, 6:13 PM

	Unknown Object (File)
	Fri, Jun 21, 11:16 AM

	Unknown Object (File)
	Mon, Jun 17, 3:42 AM

	Unknown Object (File)
	Wed, Jun 12, 2:52 PM

View All 37 Files

Subscribers

None

Details

Reviewers

michal
tomek
atul

Commits

rCOMMbded5a4c0cd2: [lib] Fix crash when logged out with sidebar open

Summary

See more details about the crash in ENG-3837. The inline code comment also provides more context on what was happening.

Test Plan

I reproduced the crash described in the Linear issue, and then confirmed it no longer occurs with this change

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ashoat created this revision.May 4 2023, 9:59 AM

Harbormaster completed remote builds in B19049: Diff 26081.May 4 2023, 10:17 AM

ashoat requested review of this revision.May 4 2023, 10:17 AM

atul accepted this revision.May 4 2023, 10:22 AM

This revision is now accepted and ready to land.May 4 2023, 10:22 AM

Closed by commit rCOMMbded5a4c0cd2: [lib] Fix crash when logged out with sidebar open (authored by ashoat). · Explain WhyMay 4 2023, 10:23 AM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMMbded5a4c0cd2: [lib] Fix crash when logged out with sidebar open.

Revision Contents
Changeset List

Path

Size

lib/

shared/

mention-utils.js

11 lines

Diff 26084

View Options

lib/shared/mention-utils.js

[lib] Fix crash when logged out with sidebar openClosedPublicActions