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

Meeting Agenda October 10, 2023 #1905

Closed
Piyopiyo-Kitsune opened this issue Oct 6, 2023 · 3 comments
Closed

Meeting Agenda October 10, 2023 #1905

Piyopiyo-Kitsune opened this issue Oct 6, 2023 · 3 comments
Assignees
Labels
[Admin] Agenda Agenda for weekly Training Team meetings.

Comments

@Piyopiyo-Kitsune
Copy link
Collaborator

Piyopiyo-Kitsune commented Oct 6, 2023

Training Team meetings start at 07:00 UTC weekly on Tuesdays. Anyone is welcome to contribute to the meeting by commenting in Slack threads throughout the week. Meeting notes will be published at the end of the week to also incorporate these asynchronous conversations.

This week’s meeting will be hosted by @Piyopiyo-Kitsune

1. Intro/Welcome

Welcome to all new teammates who joined the Training Team this week!

2. News

1. Meeting Note Takers

2. Looking for feedback

3. Looking for volunteers

4. Updates from last week's dev-squad triage session

This week the dev-squad triaged 2 new PRs and one new bug:

There was some discussion around #1880, which we would like to bring to the team.

Generally, when implementing internal anchor link fragments (where an HTML anchor tag links to the id attribute of another element on the same page instead of an external URL) the fragment uses lowercase, not sentence case. While this does not seem to be a requirement according to the HTML spec, it's not something commonly used, as most developers will use lowercase when defining id attributes of HTML elements.

However, the Tip box in the lesson plan saved pattern defines the links as sentence-case, but there are also some instances where it might be all lowercase. So while we're not sure that the case ultimately matters, we do think it should be formalized, documented, and then used accordingly across the site.

So this leaves us with a question, and a follow-up problem:

  1. Should we stick to sentence case, or switch to all lowercase.
  2. Whatever we decide, once implemented, we'll need to review both the lesson plan saved pattern, and any existing lesson plans, to ensure the links work as expected.

Therefore we'd like to bring this to the training team to discuss, decide on, and implement a plan to get the relevant parts updated.

5. Other news

3. Open requests for review

See our Guidelines for reviewing content to review the following content.

4. Project updates

5. Open Discussions

If you have topics you’d liked discussed in the meeting, please leave them as a comment on this issue.


You can see all meetings scheduled on this meeting calendar. If you are new to the Training Team, then come walk through our onboarding program to get to know the team and how we work. And if you have questions, feel free to reach out in the #training Slack channel at any time.

@Piyopiyo-Kitsune Piyopiyo-Kitsune added the [Admin] Agenda Agenda for weekly Training Team meetings. label Oct 6, 2023
@Piyopiyo-Kitsune Piyopiyo-Kitsune self-assigned this Oct 6, 2023
@Piyopiyo-Kitsune Piyopiyo-Kitsune moved this from Project Ideas to In Progress in Training Team Administration Oct 6, 2023
@jonathanbossenger
Copy link
Collaborator

Thank you @Piyopiyo-Kitsune. Notes from the dev-squad triage session have been added, with a new discussion item. You are welcome to move the discussion item to the open discussions part of the meeting, or wherever you prefer.

@jonathanbossenger
Copy link
Collaborator

Discussion item: I have created the PR for the Sensei Pro upgrade. I would like to ask any members of the faculty dev-squad, or any other contributors with developer experience, to help review and test the PR.

#1909

@webtechpooja
Copy link
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Admin] Agenda Agenda for weekly Training Team meetings.
Projects
No open projects
Development

No branches or pull requests

3 participants