Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: participants follow facilitator #7269

Merged
merged 1 commit into from
Oct 12, 2022
Merged

Conversation

mattkrick
Copy link
Member

@mattkrick mattkrick commented Oct 11, 2022

Description

Fixes #7268
Reverts #7238

Demo

[If possible, please include a screenshot or gif/video, it'll make it easier for reviewers to understand the scope of the changes and how the change is supposed to work. If you're introducing something new or changing the existing patterns, please share a Loom and explain what decisions you've made and under what circumstances]

Testing scenarios

  • Create a check-in meeting with 2 agenda items. goto item 1
    • on device 1 (facilitator) moves to item 2
    • on device 2 (participant) see that you do not advance forward

Signed-off-by: Matt Krick <matt.krick@gmail.com>
@mattkrick mattkrick changed the base branch from staging to production October 11, 2022 18:10
@mattkrick mattkrick requested a review from JimmyLv October 11, 2022 18:10
@mattkrick
Copy link
Member Author

mattkrick commented Oct 11, 2022

@JimmyLv since we're so close to shipping the next release, could you please merge this in with the other PRs? If so, just change the base branch to staging. If we aren't going to ship in the next day, just let me know & we can merge this one separately.

cc @Dschoordsch since we were chatting about this one

Copy link
Contributor

@JimmyLv JimmyLv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! will add to today's weekly release.

I merged this PR to the master branch directly, because the staging will come from master and the release-please bot helps generate the versioning.

@JimmyLv JimmyLv changed the base branch from production to master October 12, 2022 08:33
@JimmyLv JimmyLv merged commit 1800d0f into master Oct 12, 2022
@JimmyLv JimmyLv deleted the hotfix/follow-facilitator branch October 12, 2022 08:33
@github-actions github-actions bot mentioned this pull request Oct 12, 2022
JimmyLv added a commit that referenced this pull request Oct 12, 2022
* fix: other tabs break when accepting a team invitation via a link (#7200)

* fix(noImplicitAny): Fixup 200 ts rules (Part 2 of 2) (#7193)

* dx: begin client noImplicitAny

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* fix: 50 file checkpoint

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* remove getInProxy

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* finish all bugs in mutations dir

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* fix: finish remaining lint

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* turn on noImplicitAny for repo

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* address CR comments

Signed-off-by: Matt Krick <matt.krick@gmail.com>

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* chore: bootstrap release-release for automatic changelog & version changes (#7209)

* ci: bootstrap release-please

* ci: also add staging as the branch to trigger

* ci: add release-hotfix for production branch

* fix: don't include component in released Git tag

* fix: update the boostrap version & git hash

* fix: unsubscribe analytics bug (#7255)

* make subscribeToSummaryEmail an object in analytics

* second commit for pr rules

* update track type

* feat(metrics): Send isPatient0 property to Google Analytics (#7261)

* fix: participants follow facilitator (#7269)

Signed-off-by: Matt Krick <matt.krick@gmail.com>

Signed-off-by: Matt Krick <matt.krick@gmail.com>

* chore: release v6.78.0

Signed-off-by: Matt Krick <matt.krick@gmail.com>
Co-authored-by: Igor Lesnenko <igor.lesnenko@gmail.com>
Co-authored-by: Matt Krick <matt.krick@gmail.com>
Co-authored-by: JimmyLv_吕立青 <jimmy.jinglv@gmail.com>
Co-authored-by: Nick O'Ferrall <nickoferrall@gmail.com>
Co-authored-by: Bruce Tian <tianrunhe@gmail.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions github-actions bot mentioned this pull request Oct 12, 2022
@JimmyLv JimmyLv mentioned this pull request Oct 12, 2022
20 tasks
This was referenced Oct 21, 2022
jmtaber129 added a commit that referenced this pull request Feb 15, 2023
jmtaber129 added a commit that referenced this pull request Feb 21, 2023
…ed (#7779)

* Revert "fix: participants follow facilitator (#7269)"

This reverts commit 1800d0f.

* Convert DiscussionThreadInput to useFragment

* Limit discussion autofocus to standups

* CR: Remove unnecessary hook dep
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Participants don't follow facilitator in agendaItems/discuss phase
2 participants