[keyserver] Convert flag to a boolean
ClosedPublic
Actions

Authored by tomek on Tue, May 23, 6:04 AM.

Details

Reviewers

bartek
kamil
inka
michal

Commits

rCOMM4f0eb1d7c7a6: [keyserver] Convert flag to a boolean

Summary

Instead of returning a boolean we were returning a number. Everything worked correctly, because 0 is falsy. Only recently introduced output validator made the bug evident.

Test Plan

Check if fetching invite links works.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tomek created this revision.Tue, May 23, 6:04 AM

Herald added subscribers: atul, ashoat. · View Herald TranscriptTue, May 23, 6:04 AM

tomek added a parent revision: D7876: [lib] Update invite link type.Tue, May 23, 6:06 AM

tomek added a child revision: D7877: [lib] Create a store in redux for the invite links.

Harbormaster completed remote builds in B19666: Diff 26874.Tue, May 23, 6:23 AM

tomek requested review of this revision.Tue, May 23, 6:23 AM

michal accepted this revision.Tue, May 23, 7:12 AM

This revision is now accepted and ready to land.Tue, May 23, 7:12 AM

Closed by commit rCOMM4f0eb1d7c7a6: [keyserver] Convert flag to a boolean (authored by tomek). · Explain WhyWed, May 24, 4:23 AM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMM4f0eb1d7c7a6: [keyserver] Convert flag to a boolean.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

fetchers/

link-fetchers.js

2 lines

Diff 26962

View Options

keyserver/src/fetchers/link-fetchers.js

[keyserver] Convert flag to a booleanClosedPublicActions