Skip to content

fix: update client header to use <h3> instead of <h2>#179

Merged
alicejli merged 1 commit intomainfrom
updateCloudRadHeader
May 4, 2023
Merged

fix: update client header to use <h3> instead of <h2>#179
alicejli merged 1 commit intomainfrom
updateCloudRadHeader

Conversation

@alicejli
Copy link
Copy Markdown
Contributor

@alicejli alicejli commented Apr 25, 2023

Fixes #178 (part 1 of full fix)

Part 2 of fix: googleapis/doc-pipeline#436

@alicejli alicejli marked this pull request as ready for review April 26, 2023 17:43
@alicejli alicejli requested review from a team and burkedavison April 26, 2023 17:43
@burkedavison
Copy link
Copy Markdown
Member

@alicejli : Can you clarify which doc-pipeline template file is going to be modified in part 2?

@alicejli
Copy link
Copy Markdown
Contributor Author

@alicejli : Can you clarify which doc-pipeline template file is going to be modified in part 2?

@burkedavison This is the PR that updates doc-pipeline: googleapis/doc-pipeline#439. I ended up adding a new template for Java Overviews specifically.

Note that it fails the golden tests right now and @dandhlee is helping me look into how to update the goldens as the instructions don't currently work for my setup for some reason.

@alicejli alicejli merged commit b67fff3 into main May 4, 2023
@alicejli alicejli deleted the updateCloudRadHeader branch May 4, 2023 14:53
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.

Fix heading indentation on righthand nav in c.g.c. reference docs for java Fix heading indentation on righthand nav in c.g.c. reference docs

2 participants