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
nixos/chromium: update link in docs #93794
Conversation
CC person who did a similar fix @davidak |
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.
looks good
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.
I think https://cloud.google.com/docs/chrome-enterprise/policies would be better. At least for me https://cloud.google.com/docs/chrome-enterprise/policies/?policy=ExtensionInstallForcelist">ExtensionInstallForcelist doesn't seem to render correctly / contains less "extras".
The correct link is https://cloud.google.com/docs/chrome-enterprise/policies/?policy=ExtensionInstallForcelist without the extra The argument I am using Chromium Version 85.0.4183.121 (Developer Build) (64-bit) and it seems to render fine. What browser are you using? |
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.
The correct link is https://cloud.google.com/docs/chrome-enterprise/policies/?policy=ExtensionInstallForcelist without the extra ">ExtensionInstallForcelist.
Ups, sorry about that, I was stupid and ignored XML while copying the link. That also explains the strange ">
then 🤦
My bad. Thanks for the fix :)
(cherry picked from commit 0da7593)
Motivation for this change
The linked documentation states the following.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)