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.

Sign up for GitHub

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

Jump to bottom

Correct meeting minutes 2024-02-28 #716

Merged
merged 2 commits into from
Apr 24, 2024
Merged

Correct meeting minutes 2024-02-28 #716

merged 2 commits into from
Apr 24, 2024

Conversation

tschmidtb51
Copy link
Contributor

  • try to unify format
  • add questions to unclear points
  • improve wording from memory

@tschmidtb51
Correct meeting minutes 2024-02-28
a17d113 
- try to unify format
- add questions to unclear points
- improve wording from memory
@tschmidtb51 tschmidtb51 self-assigned this Apr 3, 2024
@tschmidtb51 tschmidtb51 added the meeting_minutes TC meeting minutes label Apr 3, 2024
tschmidtb51
tschmidtb51 commented Apr 3, 2024
View reviewed changes
meeting_minutes/2024/2024-02-28.md Outdated
- TC should fix in 2.1 or another version.
- For current implementations, a router may be needed.
- For current implementations, a router <!-- Just marking here that this makes no sense to me. Maybe it should refer to an errata? --> may be needed.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a point for discussion - I think errata might be the correct wording instead of "router"

Copy link
Contributor

@sthagen sthagen Apr 3, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

Close captioning for precedence 😁
Yes, having observed Webex close captioning closely, I assume this is a correct hypothesis:

a router → errata

sthagen
sthagen approved these changes Apr 3, 2024
View reviewed changes
Copy link
Contributor

@sthagen sthagen left a comment

Choose a reason for hiding this comment

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

LGTM - a tad over engineered maybe, but February is long gone.

santosomar
santosomar approved these changes Apr 3, 2024
View reviewed changes
Copy link
Contributor

@santosomar santosomar left a comment

Choose a reason for hiding this comment

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

Thank you so much for catching those!

@tschmidtb51 tschmidtb51 marked this pull request as ready for review April 24, 2024 16:33
@santosomar santosomar merged commit 3df3076 into oasis-tcs:master Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sthagen sthagen sthagen approved these changes

@santosomar santosomar santosomar approved these changes

Assignees

@tschmidtb51 tschmidtb51

Labels
meeting_minutes TC meeting minutes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tschmidtb51 @sthagen @santosomar