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

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

Merged
merged 1 commit into from
May 4, 2023

Conversation

alicejli
Copy link
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 a review from a team as a code owner April 26, 2023 17:43
@alicejli alicejli requested a review from burkedavison April 26, 2023 17:45
@burkedavison
Copy link
Member

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

@alicejli
Copy link
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
2 participants