[lib] Fix an invalid condition
ClosedPublic
Actions

Authored by tomek on Jul 16 2024, 6:14 AM.

Details

Reviewers

kamil
marcin

Commits

rCOMM5eba6c1c763d: [lib] Fix an invalid condition

Summary

We should check the accessToken instead of checking userID twice.

Depends on D12749

Test Plan

Flow

Diff Detail

Repository

rCOMM Comm

Branch

dms (branched from palys-swm/dms)

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

tomek created this revision.Jul 16 2024, 6:14 AM

Herald added a subscriber: ashoat. · View Herald TranscriptJul 16 2024, 6:14 AM

Harbormaster completed remote builds in B30383: Diff 42346.Jul 16 2024, 6:31 AM

tomek requested review of this revision.Jul 16 2024, 6:31 AM

tomek added a child revision: D12776: [lib] Generate P2P messages based on actions.Jul 16 2024, 7:24 AM

🤦‍♂️🙄
Thanks for fixing

This revision is now accepted and ready to land.Jul 18 2024, 6:44 AM

tomek edited parent revisions, added: D12747: [lib] Enable automatic retries for some messages; removed: D12749: [lib] Stop requiring sequential messages.Jul 18 2024, 7:10 AM

Closed by commit rCOMM5eba6c1c763d: [lib] Fix an invalid condition (authored by tomek). · Explain WhyJul 19 2024, 7:47 AM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMM5eba6c1c763d: [lib] Fix an invalid condition.

Revision Contents
Changeset List

Path

Size

lib/

tunnelbroker/

peer-to-peer-context.js

6 lines

Diff 42346

View Options

lib/tunnelbroker/peer-to-peer-context.js

This file was added.

				// @flow

				import * as React from 'react';

				import type { OLMIdentityKeys } from '../types/crypto-types.js';
				import type { OlmSessionInitializationInfo } from '../types/request-types.js';

				export type NotificationsSessionCreatorContextType = {
				+notificationsSessionCreator: (
				cookie: ?string,
				notificationsIdentityKeys: OLMIdentityKeys,
				notificationsInitializationInfo: OlmSessionInitializationInfo,
				deviceID: string,
				) => Promise<string>,
				};

				const NotificationsSessionCreatorContext: React.Context<?NotificationsSessionCreatorContextType> =
				React.createContext(null);

				export { NotificationsSessionCreatorContext };

[lib] Fix an invalid conditionClosedPublicActions