add_metis_calendars_to_linkal #8

mdebray · 2023-10-03T01:14:31+02:00

mdebray commented

2023-10-03 01:14:31 +02:00

Bon le script jq est moche. On pourrait le remplacer par du nix. Normalement ça ~marche (j'ai pas pu tester complètement dans la vm à cause d'histoires de réseau et de dns)

mdebray added 1 commit

2023-10-03 01:14:31 +02:00

fix(metis): Use serverAliases for metis 143bc82d3e

mdebray force-pushed add_metis_calendars_to_linkal from b9c8214803 to 0ee8760f8d

2023-10-03 01:18:09 +02:00

Compare

thubrecht reviewed

2023-10-03 09:38:29 +02:00

machines/web01/linkal/module.nix

					
				@ -2,3 +2,3 @@

				let

				  inherit (lib) mapAttrs' mkEnableOption mkIf mkOption nameValuePair types;

				  inherit (lib) mapAttrs mapAttrs' mkEnableOption mkIf mkOption nameValuePair types;

mapAttrs est un builtin depuis nix 2.1, en plus t'as pas l'air de l'utiliser ^^

`mapAttrs` est un builtin depuis nix 2.1, en plus t'as pas l'air de l'utiliser ^^

Effectivement

mdebray marked this conversation as resolved

thubrecht reviewed

2023-10-03 09:38:31 +02:00

machines/web01/linkal/module.nix

					
				@ -28,0 +28,4 @@

				          config = mkOption {

				            type = path;

				            default = jsonFormat.generate "linkal-config.json" { inherit (config) calendars; };

				            internal = true;

Pourquoi internal si l'objectif c'est de pouvoir override le fichier de config ?

Hmm l'idée était de dire c'est une interface uniquement pour les utilisateurs avancés mais en vrai on peut l'enlever

thubrecht reviewed

2023-10-03 09:40:13 +02:00

machines/web01/metis/default.nix

					
				@ -32,6 +25,11 @@ let

				  };

				in {

				  _module.args.metisProviders = {

Hmmm, je suis pas certain que de passer metisProvider à tous les modules soit une super idée, ce serait plus propre de laisser metis definir ses providers et de les récupérer avec (import sources.metis { ... }).providers

Hmmm, je suis pas certain que de passer `metisProvider` à tous les modules soit une super idée, ce serait plus propre de laisser metis definir ses providers et de les récupérer avec `(import sources.metis { ... }).providers`