Skip to content

Conversation

@Eveeifyeve
Copy link

Continued pr.

Closes #12920.
Added @nenikitov as co author

feat: add managed schema for chrome

fix: firefox warning about unknown key storage in manifest

chore: format and sync new settings

wip: make managed settings take priority

docs: policy settings

feat(settings): add policy storage for firefox
@Eveeifyeve Eveeifyeve changed the title Open [Feature] Support preconfiguration via Policy continued [Feature] Support preconfiguration via Policy continued Jun 27, 2025
@Eveeifyeve Eveeifyeve marked this pull request as ready for review June 27, 2025 09:01
@Eveeifyeve
Copy link
Author

I will be happy to provide a commit for nix dirvation for testing before merging remove that commit.

@Eveeifyeve
Copy link
Author

CC @alexanderby @darkreaderdev for review.

@voronind-com
Copy link

Aight, here we go to wait another year here

@Eveeifyeve
Copy link
Author

dw I will ping those people every month, because this just can't be waited to be reviewed. And nix users have a default.nix to use for testing.

@Myshor
Copy link
Collaborator

Myshor commented Jul 2, 2025

I won't touch anything here but I can request review from Alexander - so one more ping will be sent to him. 😉

@Myshor Myshor requested a review from alexanderby July 2, 2025 14:37
@SolninjaA
Copy link

Just FYI, there's a typo in default.nix. overrideAtrs should be overrideAttrs.

Also, I can't get the default.nix to work on my machine, so I am looking forward to when this gets merged!

@nenikitov
Copy link

nenikitov commented Oct 22, 2025

It seems Home Manager added extension settings through programs.firefox.profiles.<name>.extensions.settings.<name>.settings, which works without policy support.

This setting only supports storage.local, so syncSettings = false is required.

"addon@darkreader.org".settings = {
  syncSettings = false;
};

Plus, it only works if you installed extensions with programs.firefox.profiles.<name>.extensions.packages and not with programs.firefox.policies (for that this PR is needed).

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.

5 participants