-
Notifications
You must be signed in to change notification settings - Fork 132
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
Page details: updating doc to new format #2263
Conversation
Garneauma
commented
Sep 27, 2023
- Updating doc to new data-first format
- Minor PFT changes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Incomplete review.
There is a few changes and notes. Let's discuss next week.
0526cf3
to
9b6ea21
Compare
9b6ea21
to
a900ff7
Compare
Pre-approved upon review + local testing This is only a patch for GCWeb Jekyll, no impact on GCWeb versioning API, only content change for GCWeb. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Incomplete review
a900ff7
to
741354a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed and tested locally,
Other than the small change, it will be good to merge once completed.
Side note: Regarding the property "assets" in the iteration, we might split it into 2 properties like "Code sample" + "Assets" but let keep that change for later.
741354a
to
57253c6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One little change, once completed we will be good to merge this PR.
57253c6
to
7d4f40e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and reviewed.
This is only a gcweb-jekyll patch because it did touch some includes file where their output didn't change. There is no impact regarding the GCWeb versionning API.