Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Add config option to auto-join new users to rooms #2545

Merged
merged 3 commits into from
Oct 17, 2017
Merged

Conversation

dbkr
Copy link
Member

@dbkr dbkr commented Oct 16, 2017

New users who register on the server will be dumped into all rooms in
auto_join_rooms in the config.

New users who register on the server will be dumped into all rooms in
auto_join_rooms in the config.
@turt2live
Copy link
Member

ooi, how does this react with riot-bot?

@dbkr
Copy link
Member Author

dbkr commented Oct 16, 2017

riot-bot is done on the client and creates a new room each time, so I guess will continue to be on the client side - other than that, they'll probably stay fairly separate.

@dbkr dbkr merged commit 4e242b3 into develop Oct 17, 2017
@maxidorius
Copy link
Contributor

Could this be moved somewhere reachable by password providers? Else this is only available for users that directly register on the homeserver and not to users within LDAP on first login, etc.

@alexmoras
Copy link

+1 on what @maxidor said. I'm running Matrix/Riot in an environment where it would be very helpful to have these autojoin config options work with LDAP. For reference, I'm using the LDAP password providers method not MXISD.

@ghost ghost mentioned this pull request Dec 2, 2017
@Ezwen Ezwen mentioned this pull request Feb 23, 2018
@hawkowl hawkowl deleted the dbkr/auto_join_rooms branch September 20, 2018 14:02
@ToddServo
Copy link

So auto-join rooms on registration is removed?
I updated my docker file today. I was greeted with a weird static page [https://github.com//pull/4289]

Synapse's registration worked great.
Riot is ok, but the signup page is not as mobile friendly -and- lacks the ability to auto-join new users to a room.
I've had to switch to eeacms's synapse docker... and I'm sure they're thinking the same thing I was. "What"

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

Successfully merging this pull request may close these issues.

6 participants