Skip to content
This repository has been archived by the owner on Aug 10, 2023. It is now read-only.

The room must be public to use this integration #464

Open
HarHarLinks opened this issue Jan 18, 2022 · 1 comment
Open

The room must be public to use this integration #464

HarHarLinks opened this issue Jan 18, 2022 · 1 comment

Comments

@HarHarLinks
Copy link

HarHarLinks commented Jan 18, 2022

When trying to configure hookshot. Why would it need to be public? Is it for the dimension user to do things?

The bridge can be configured "manually" without the room being public. It would be good if Dimension could be used as UI for provisioning in these cases also.

@yncyrydybyl
Copy link

yncyrydybyl commented Apr 30, 2022

It think it because of

export class Bridge extends Integration {
constructor(bridge: BridgeRecord, public config: any) {
super(bridge);
this.category = "bridge";
if (PRIVATE_ACCESS_SUPPORTED_BRIDGES.indexOf(bridge.type) !== -1) this.requirements = [];
else this.requirements = [{
condition: "publicRoom",
expectedValue: true,
argument: null, // not used
}];
// We'll just say we don't support encryption
this.isEncryptionSupported = false;
}
}

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants