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

Handle per-semester parents in Quest #413

Merged
merged 7 commits into from
Sep 26, 2024

Conversation

BillWagner
Copy link
Member

There are three sets of changes here:

  1. A bit of refactoring to keep the communications with Azure DevOps in one place.
  2. Expand the config object to include semesters on any parentNode object. Now, the semester and Label tags are both optional:
    • If Label is null, the parentNode becomes the default node for that semester.
    • if semester is null, this parentNode is used for all semesters. This is primarily for backwards compatibility
  3. Read the updated config and use that info to set the correct parent.

BillWagner and others added 7 commits September 26, 2024 15:26
This reverts commit 8e028fd45dd4b3ec35289dc60f1d7f303cf08325.
Move some code around to make it easier to add the config features without extensive copying.
Set the parent based on semester + parent config.
Co-authored-by: David Pine <david.pine@microsoft.com>
@BillWagner BillWagner enabled auto-merge (squash) September 26, 2024 19:26
@BillWagner BillWagner merged commit 22d0f23 into dotnet:main Sep 26, 2024
8 checks passed
@BillWagner BillWagner deleted the semester-change branch September 26, 2024 19:27
BillWagner added a commit to BillWagner/docs that referenced this pull request Sep 26, 2024
dotnet/docs-tools#413 updated Sequester to handle parents per semester.

This updates the config to respond to those changes.
BillWagner added a commit to BillWagner/AspNetCore.Docs that referenced this pull request Sep 26, 2024
dotnet/docs-tools#413 enhances the config object to support different parents per semester. This adds the updated config for that purpose.
BillWagner added a commit to BillWagner/docs-aspire that referenced this pull request Sep 26, 2024
dotnet/docs-tools#413 enhances the config object to have different parents per semester. This PR updates the config to use the correct parents in both active semesters.
BillWagner added a commit to BillWagner/docs-maui that referenced this pull request Sep 26, 2024
dotnet/docs-tools#413 enhances the Sequester config so that parents can change per semester. This PR updates the config in this repo to match
BillWagner added a commit to BillWagner/maui-samples that referenced this pull request Sep 26, 2024
dotnet/docs-tools#413 enhances Sequester to allow for different parents per semester. This PR updates the config to match.
BillWagner added a commit to BillWagner/dotnet-api-docs that referenced this pull request Sep 26, 2024
dotnet/docs-tools#413 updates Sequester to provide for different parents each semester. This PR updates the config to match.
BillWagner added a commit to dotnet/docs that referenced this pull request Sep 26, 2024
dotnet/docs-tools#413 updated Sequester to handle parents per semester.

This updates the config to respond to those changes.
BillWagner added a commit to dotnet/dotnet-api-docs that referenced this pull request Sep 26, 2024
dotnet/docs-tools#413 updates Sequester to provide for different parents each semester. This PR updates the config to match.
davidbritch pushed a commit to dotnet/maui-samples that referenced this pull request Sep 27, 2024
dotnet/docs-tools#413 enhances Sequester to allow for different parents per semester. This PR updates the config to match.
davidbritch pushed a commit to dotnet/docs-maui that referenced this pull request Sep 27, 2024
dotnet/docs-tools#413 enhances the Sequester config so that parents can change per semester. This PR updates the config in this repo to match
BillWagner added a commit to dotnet/AspNetCore.Docs that referenced this pull request Sep 27, 2024
dotnet/docs-tools#413 enhances the config object to support different parents per semester. This adds the updated config for that purpose.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants