Strange that diffs that appear to depend on this one are already landed, before this one. Will this present any issues on master? Does it block release?

In D10926#315341, @ashoat wrote:

Strange that diffs that appear to depend on this one are already landed, before this one.

D10785 and D10787 depend on this one - those are still in review.

Will this present any issues on master? Does it block release?

No, it's just a refactor - logic is the same.

inka accepted this revision.Feb 5 2024, 10:28 AM

This revision is now accepted and ready to land.Feb 5 2024, 10:28 AM

Closed by commit rCOMM4fa3ac20d292: [web] refactor set new session in `redux-setup` to ops. · Explain WhyFeb 5 2024, 6:11 PM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM4fa3ac20d292: [web] refactor set new session in `redux-setup` to ops.

Revision Contents
Changeset List

Path

Size

web/

redux/

redux-setup.js

21 lines

Diff 36630

View Options

[web] refactor set new session in `redux-setup` to opsClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 36630

web/redux/redux-setup.js

[web] refactor set new session in `redux-setup` to ops
ClosedPublic
Actions

Revision Contents
Changeset List