Update with smaller routes file #166
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…nfigs 📝 (publish-redirects.html): comment out unused redirect logic for clarity The BuildApi job is temporarily disabled by setting its condition to false, likely for testing or maintenance purposes. The jq command is corrected to include the $routesConfig file, ensuring all necessary configurations are merged. In the publish-redirects.html file, the redirect logic is commented out, indicating it is not currently needed but preserved for potential future use. This improves the clarity and maintainability of the code.
…eadability The comment block formatting is adjusted to ensure that the closing comment tag is aligned correctly. This change improves the readability and maintainability of the code by ensuring that the comment structure is clear and consistent.
The BuildApi step is removed from the dependencies of the Publish Site job. This change is made because the API build is no longer required for the site publishing process, streamlining the workflow and reducing unnecessary steps.
…workflow
The download of a single artifact is now conditionally disabled by
setting the `if` condition to `${{ false }}`. This change is made to
prevent unnecessary downloading of artifacts during the workflow
execution, which can save time and resources if the artifact is not
needed for the current workflow run.
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://yellow-pond-042d21b03-166.westeurope.5.azurestaticapps.net |
The route and redirect mappings were reversed, causing incorrect redirect behavior. By swapping the values, the correct URL is now redirected to the intended target, ensuring users are directed to the correct course pages. This change resolves navigation issues and improves user experience.
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://yellow-pond-042d21b03-166.westeurope.5.azurestaticapps.net |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🔧 (main.yaml): disable BuildApi job and fix jq command for merging configs
📝 (publish-redirects.html): comment out unused redirect logic for clarity
The BuildApi job is temporarily disabled by setting its condition to false, likely for testing or maintenance purposes. The jq command is corrected to include the $routesConfig file, ensuring all necessary configurations are merged. In the publish-redirects.html file, the redirect logic is commented out, indicating it is not currently needed but preserved for potential future use. This improves the clarity and maintainability of the code.