Merge pull request #1598 from matrix-org/luke/groups-update-group-room-visibility-api
Modify the group room visibility API to reflect the js-sdk changespull/21833/head
						commit
						7893927e14
					
				| 
						 | 
				
			
			@ -137,7 +137,7 @@ module.exports = React.createClass({
 | 
			
		|||
        const groupId = this.props.groupId;
 | 
			
		||||
        const roomId = this.props.groupRoomId;
 | 
			
		||||
        const roomName = this.state.groupRoom.displayname;
 | 
			
		||||
        this._groupStore.updateGroupRoomAssociation(roomId, isPublic).catch((err) => {
 | 
			
		||||
        this._groupStore.updateGroupRoomVisibility(roomId, isPublic).catch((err) => {
 | 
			
		||||
            console.error(`Error whilst changing visibility of ${roomId} in ${groupId} to ${isPublic}`, err);
 | 
			
		||||
            const ErrorDialog = sdk.getComponent("dialogs.ErrorDialog");
 | 
			
		||||
            Modal.createTrackedDialog('Failed to remove room from group', '', ErrorDialog, {
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -150,9 +150,9 @@ export default class GroupStore extends EventEmitter {
 | 
			
		|||
            .then(this._fetchRooms.bind(this));
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    updateGroupRoomAssociation(roomId, isPublic) {
 | 
			
		||||
    updateGroupRoomVisibility(roomId, isPublic) {
 | 
			
		||||
        return this._matrixClient
 | 
			
		||||
            .updateGroupRoomAssociation(this.groupId, roomId, isPublic)
 | 
			
		||||
            .updateGroupRoomVisibility(this.groupId, roomId, isPublic)
 | 
			
		||||
            .then(this._fetchRooms.bind(this));
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in New Issue