Skip to content

Vue.js create room flow

Eric Eastwood requested to merge vue-create-room-flow into develop

Vue.js create room flow.

One big change is that you can now create a room and associate it with any repo you have admin access to. Previously repo association in the UI only happened if the URI matched directly with GitHub. This wasn't a backend change so was possible before in the API. I think it feels a lot nicer to be able to associate your repo with any room and makes sense to allow this to gear up for the GitHub URI split.

Part of https://gitlab.com/gitlab-org/gitter/webapp/issues/2411

MR to remove old create room code: https://gitlab.com/gitlab-org/gitter/webapp/-/merge_requests/1831

. . . .
Errors . .
A6Xqmwq .

Testing strategy

  1. Play with the modal in general
  2. Create a public room
  3. Create a private room
  4. Create a room associated with a GitHub repo
  5. Create a room associated with a GitLab project

The modal state does not reset between closing and opening. Not totally against resetting but thought it was useful so you can go out, get help with your OAuth problems and jump back in. It does re-fetch your admin groups and repos as those may have changed in the time frame.

Todo

Edited by Eric Eastwood

Merge request reports