Page MenuHomePhabricator
Differential D10843

[lib] Clear connection issue on successful login
ClosedPublic
Actions

Authored by tomek on Jan 26 2024, 8:51 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 17, 1:30 PM
Unknown Object (File)
Feb 25 2025, 4:03 AM
Unknown Object (File)
Feb 20 2025, 6:40 PM
Unknown Object (File)
Feb 20 2025, 4:17 AM
Unknown Object (File)
Feb 18 2025, 10:30 PM
Unknown Object (File)
Feb 10 2025, 12:00 AM
Unknown Object (File)
Feb 9 2025, 11:59 PM
Unknown Object (File)
Feb 7 2025, 8:12 PM
View All 83 Files
Subscribers
ashoat

Details

Reviewers
inka
kamil
varun
Commits
rCOMMee45a9d54c10: [lib] Clear connection issue on successful login
Summary

After successful auth we can conclude that a connection issue no longer exists

https://linear.app/comm/issue/ENG-6027/reset-connectionissue-field

Depends on D10842

Test Plan

Log out, dispatch an action that sets a connection issue, log in - the connection issue should be cleared.

Diff Detail

Repository
rCOMM Comm
Branch
handlers-2
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

tomek created this revision.Jan 26 2024, 8:51 AM
Herald added a subscriber: ashoat. · View Herald TranscriptJan 26 2024, 8:51 AM
Harbormaster returned this revision to the author for changes because remote builds failed.Jan 26 2024, 8:57 AM
Harbormaster failed remote builds in B26198: Diff 36186!
tomek added a child revision: D10844: [lib] Retry auth.Jan 26 2024, 9:12 AM
Harbormaster completed remote builds in B26198: Diff 36186.Jan 26 2024, 9:16 AM
tomek requested review of this revision.Jan 26 2024, 9:16 AM
varun resigned from this revision.Jan 29 2024, 7:51 PM

looks fine to me but someone else should take a look

inka accepted this revision.Jan 30 2024, 1:09 AM
This revision is now accepted and ready to land.Jan 30 2024, 1:09 AM
tomek updated this revision to Diff 36533.Feb 1 2024, 9:08 AM

Rebase

Harbormaster completed remote builds in B26422: Diff 36533.Feb 1 2024, 9:27 AM
Closed by commit rCOMMee45a9d54c10: [lib] Clear connection issue on successful login (authored by tomek). · Explain WhyFeb 5 2024, 7:23 AM
This revision was automatically updated to reflect the committed changes.
tomek added a commit: rCOMMee45a9d54c10: [lib] Clear connection issue on successful login.

Revision Contents
Changeset List

PathSize
lib/
reducers/
4 lines

Diff 36186

View Options

lib/reducers/keyserver-reducer.js

This file contains generated code, which does not normally need to be reviewed. Show File Contents