Skip to content

Conversation

@daspecster
Copy link
Contributor

@daspecster daspecster commented Apr 22, 2016

This allows the side-nav background color to extend
to the bottom of the screen.

Before:
side-nav-before

After:
side-nav-after

This allows the side-nav background color to extend
to the bottom of the screen.
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 22, 2016
@daspecster daspecster changed the title Remove background: none; Fix side-nav background. Apr 22, 2016
@daspecster daspecster changed the title Fix side-nav background. Fix docs side-nav background. Apr 22, 2016
@dhermes
Copy link
Contributor

dhermes commented Apr 22, 2016

LGTM. FWIW we've had #1704 and #1028 for this but both were closed since we will be ditching this template once the generic gcloud-common template is in use.

@daspecster
Copy link
Contributor Author

Ah, I should have searched. Thanks @dhermes!

Should I merge or you? I'm in practice of not merging my own code for peer reviewing.

@dhermes
Copy link
Contributor

dhermes commented Apr 22, 2016

When it is amongst @tseaver and myself, the author does the merge. (In causes where a contributor can't merge, then the reviewer does.)

@daspecster daspecster merged commit 69f481b into googleapis:master Apr 22, 2016
@daspecster daspecster deleted the fix-docs-left-nav-background branch April 22, 2016 02:19
parthea pushed a commit that referenced this pull request Nov 26, 2025
* feat: add functionality to hash data (#1677)

* feat: add functionality to hash data

* change sensitive fields to private

* update to sha512

* update docstring

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* chore: add request-response log helpers (#1685)

* chore: add request-response log helpers

* fix presubmit

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* feat: opt-in logging support for request / response (#1686)

* feat: opt-in logging support for request/response

* add pragma no cover

* add test coverage for request/response

* add code coverage

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* chore: remove logging for async requests (#1698)

* chore: remove logging for async requests

* change Dict to Mapping

* fix mypy and lint issues

* address PR feedback

* link issue

* feat: parse request/response for logging (#1696)

* feat: parse request/response for logging

* add test case for list

* address PR comments

* address PR feedback

* fix typo

* add test coverage

* add code coverage

* feat: hash sensitive info in logs (#1700)

* feat: hash sensitive info in logs

* make helper private

* add code coverage

* address PR feedback

* fix mypy type issue

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* feat: add support for async response log (#1733)

* feat: add support for async response log

* fix whitespace

* add await

* add code coverage

* fix lint

* fix lint issues

* address PR feedback

* address PR feedback

* link issue

* feat: add request response logs for sync api calls (#1747)

* fix: remove dependency on api-core for logging (#1748)

* fix: remove dep on api-core for logging

* disable propagation to the root logger

* update async helpers tests

* fix lint issue

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants