Skip to content

Synchronize Advisory Committee representative and Consortium Member Representative #937

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
Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Synchronize Advisory Committee representative and Consortium Member Representative #937

wants to merge 4 commits into from

Conversation

@frivoal
Copy link
Collaborator

@frivoal frivoal commented Oct 25, 2024 •
edited by pr-preview bot
Loading

TallTed
TallTed reviewed Oct 28, 2024
View reviewed changes
@w3c w3c deleted a comment Nov 28, 2024
@w3c w3c deleted a comment Nov 28, 2024
tantek
tantek approved these changes Jan 16, 2025
View reviewed changes
Copy link
Contributor

@tantek tantek left a comment

Choose a reason for hiding this comment

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

This seems like a reasonable step forward.

I'd request that the editor consider the impact of these changes on the "alternate representative" role to make sure that role still works as expected/desired. I trust the editor's judgment on this interaction.

frivoal and others added 2 commits May 13, 2025 10:25
@frivoal frivoal added this to the Deferred milestone May 19, 2025
@frivoal
Copy link
Collaborator Author

frivoal commented Jul 4, 2025

The proposed text has mentions member associations, and therefore has a dependency on https://github.com/w3c/board/issues/249

TallTed
TallTed suggested changes Jul 7, 2025
View reviewed changes
Copy link
Member

@TallTed TallTed left a comment

Choose a reason for hiding this comment

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

Language tweaks for clarity.

@frivoal @TallTed
Fix syntax
c70e51b 
Co-authored-by: Ted Thibodeau Jr <[email protected]>
tantek
tantek approved these changes Jul 15, 2025
View reviewed changes
Copy link
Contributor

@tantek tantek left a comment

Choose a reason for hiding this comment

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

Looks good with either of the suggested fixes by @TallTed.

@plehegar plehegar removed this from the Deferred milestone Aug 19, 2025
@frivoal @TallTed
Adjust phrasing
a644647 
Co-authored-by: Ted Thibodeau Jr <[email protected]>
@frivoal frivoal requested a review from TallTed September 25, 2025 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

3 more reviewers

@tantek tantek tantek approved these changes

@TallTed TallTed TallTed approved these changes

@Heathoney Heathoney Heathoney approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

DO NOT MERGE: not agreed yet Needs AB Feedback Advisory Board Input needed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@frivoal @tantek @TallTed @Heathoney @plehegar