DGNum/demarches-normaliennes
15
1
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity
11991 commits 2 branches 1166 tags 86 MiB
Commit graph

11991 commits

This branch
This branch
All branches
Author SHA1 Message Date
Pierre de La Morinerie
8b5a012e14
Merge pull request #5936 from betagouv/fix-zeitwerk-smtp-warning 
Correction de messages d'avertissement liés à Zeitwerk (#5936)
 2021-02-25 11:44:52 +01:00
Pierre de La Morinerie
d36a684315 config: fix zeitwekr warning when reloading the app 
Turns out we need not only to load the Job constants later, but also
not to do the same work twice – otherwise we'll get a 

> ApiEntreprise::Job constant is already defined

when attempting to re-define the constant.
 2021-02-24 19:01:27 +01:00
Pierre de La Morinerie
9f676c76e1 config: fix zeitwerk warning about DynamicSmtpSettingsInterceptor 
Fix a warning when running tests:

> DEPRECATION WARNING: Initialization autoloaded the constant DynamicSmtpSettingsInterceptor.
>
> Being able to do this is deprecated. Autoloading during initialization is going
to be an error condition in future versions of Rails.
> 
> Reloading does not reboot the application, and therefore code executed during
> initialization does not run again. So, if you reload DynamicSmtpSettingsInterceptor, for example,
> the expected changes won't be reflected in that stale Class object.
> 
> This autoloaded constant has been unloaded.
> 
> Please, check the "Autoloading and Reloading Constants" guide for solutions.

However if we fix as recommanded, the interceptor will get added
each time the classes are reloaded. And as the actual class instance
changed after the reloading, they won't be de-duplicated – *and*
there's no way to remove the old interceptor without having a reference
to the (now-deleted) class.

Instead we load the interceptor once, and add a message about the class
not being auto-reloaded.
 2021-02-24 19:00:29 +01:00
Pierre de La Morinerie
28722b002b
Merge pull request #5932 from betagouv/sprockets-4 
Mise à jour vers le processeur d'assets Sprockets v4 (#5932)
 2021-02-24 17:56:48 +01:00
Pierre de La Morinerie
68425929c6 config: disable Sentry sampling when not enabled 
This removes spam in the debug console when running locally.

Removed messages look like a swarm of:

> [Tracing] Discarding <rails.request> transaction </assets/marianne.png> because it's not included in the random sample (sampling rate = 0.001)
 2021-02-24 17:45:18 +01:00
Pierre de La Morinerie
2e3ddd6190 gems: upgrade to Sprockets 4 2021-02-24 17:09:51 +01:00
Pierre de La Morinerie
e51948ab5c stylesheets: merge print rules with the application stylesheet 
- Having a separate stylesheet for print rules is not a best practice
  (it makes an extra network request)
- It makes migrating to Sprockets 4 easier
 2021-02-24 17:09:51 +01:00
Pierre de La Morinerie
e1a450fd6f gems: remove xray-rails 
It is not compatible with sprockets-4.
 2021-02-24 17:09:51 +01:00
Pierre de La Morinerie
2295c4200a
Merge pull request #5933 from betagouv/split-controller-tests 
CI : séparation des groupes de tests en trois
 2021-02-24 17:02:42 +01:00
Pierre de La Morinerie
3304cb3d43 ci: split controller tests into 3 groups 
Ideally we'd like:

- One group with `spec/controllers/api/**/*_spec.rb` files
- One group with everything else

But due to the way globbing works (and doesn't allow to say "Files NOT
starting with `api`"), this looks like a sensible and future-proof
configuration.
 2021-02-24 16:27:55 +01:00
Pierre de La Morinerie
4c66d2d206
Merge pull request #5931 from betagouv/app-name-initializer 
Nettoyage : déplacement de la configuration du nom de l'app dans un initialiseur (#5931)
 2021-02-24 15:47:59 +01:00
Pierre de La Morinerie
fb305e31c4 config: move application_name to an initializer 
It seems cleaner not to require a file before the Rails configuration
runs.
 2021-02-24 15:06:55 +01:00
Paul Chavard
e920764078
Merge pull request #5919 from tchak/gh-actions 
Add github actions CI
 2021-02-23 13:05:20 +01:00
Paul Chavard
a03d812d19 Add github actions CI 2021-02-23 10:35:30 +01:00
Paul Chavard
46a839e7f7 Remove a spec that was testing bug in libgeos 
we don’t use libgeos anymore
 2021-02-23 10:35:30 +01:00
Pierre de La Morinerie
b28f2e83c1
Merge pull request #5921 from betagouv/fix-zeitwerk-warning 
Suppression d'un avertissement zeitwerk au démarrage de l'app en environnement de développement local (#5921)
 2021-02-23 10:28:23 +01:00
Pierre de La Morinerie
a2175ed6e3 config: reload compatibility aliases on classes reload 
Fixes zeitwerk complaining that the compatibility aliases loaded in an
initializer will never be reloaded.

In our case it doesn't matter that much, but it will reduce the console
spam.
 2021-02-23 09:24:13 +00:00
Paul Chavard
4209253091
Merge pull request #5929 from betagouv/fix-rspec-warning 
specs: fix rspec warning about raise_error specificity
 2021-02-23 09:35:08 +01:00
Pierre de La Morinerie
d9ea415d44 specs: fix rspec warning about raise_error specificity 2021-02-23 08:25:31 +01:00
Paul Chavard
4a08f6cf3f
Merge pull request #5927 from tchak/fix-multiselect-labels 
Fix multiselect labels
 2021-02-19 14:25:28 +01:00
Paul Chavard
2c2b392d6c Fix multiselect labels 
only add values to new values if it is a new value
 2021-02-19 12:08:54 +01:00
Paul Chavard
a7331dd30d
Merge pull request #5925 from tchak/fix-constant 
fix constant
 2021-02-19 11:31:44 +01:00
Paul Chavard
92158cecb6 fix constant 2021-02-19 11:29:08 +01:00
Paul Chavard
1745f58496
Merge pull request #5923 from tchak/improuve-multi-select 
Improuve multi select
 2021-02-19 11:15:02 +01:00
Paul Chavard
24f71ccc1a Improuve multi select 
In multi select with acceptNewValues option alway keep the new value as the first item in the list to make it easier to add it
 2021-02-19 10:53:16 +01:00
Paul Chavard
20f4ca8512
Merge pull request #5838 from tchak/structured-adresse 
Structured address
 2021-02-17 19:43:18 +01:00
Paul Chavard
a0b53d7d80 Expose structured address on the API 
fix #4290
 2021-02-17 19:10:16 +01:00
Paul Chavard
75392272fb Add champ address model 2021-02-17 19:10:16 +01:00
Paul Chavard
228a83f8a5 Add api address external data fetch 2021-02-17 19:10:16 +01:00
Paul Chavard
7eee14feef Add ChampFetchExternalDataJob 2021-02-17 19:10:16 +01:00
Paul Chavard
c343fd432e
Merge pull request #5916 from betagouv/5911-allonge-conservation-dossiers 
5911 allonge conservation dossiers
 2021-02-17 18:02:15 +01:00
Christophe Robillard
ac56581ccf fix indent 2021-02-17 16:41:57 +01:00
Christophe Robillard
d55f156aec admin: durée de conservation des données hors ds n'est plus demandée 
La colonne est tout de même gardée en base
 2021-02-17 16:41:57 +01:00
Christophe Robillard
2be4bcf22d admin: peut modifier durée de conservation d'une procédure publiée 2021-02-17 16:41:57 +01:00
krichtof
79bcd28065
Merge pull request #5915 from tchak/cleanup-assets 
Remove unused paths from assets pipeline
 2021-02-17 11:32:55 +01:00
Paul Chavard
7223d77058 Remove unused paths from assets pipeline 2021-02-17 10:16:54 +01:00
krichtof
95718a3b23
Merge pull request #5913 from betagouv/remove-jquery 
supprime jquery et select2
 2021-02-16 20:20:27 +01:00
Christophe Robillard
b884ecbb4c supprime jquery et select2 2021-02-16 18:36:42 +01:00
Paul Chavard
7ca89f9bb0
Merge pull request #5910 from tchak/update-react 
Update react
 2021-02-16 18:30:12 +01:00
Paul Chavard
6756db1013 Update react-query 2021-02-16 18:14:11 +01:00
Paul Chavard
bb49f6e151 replace react-use with use-debounce 
this is a much lighter dependencie
 2021-02-16 18:14:10 +01:00
Paul Chavard
3dae5b5c04 update react 2021-02-16 18:14:10 +01:00
Paul Chavard
df80f5681d remove react-scroll-to-component 2021-02-16 18:14:10 +01:00
Paul Chavard
1c164c4376 Use jsx extension 2021-02-16 18:14:10 +01:00
Pierre de La Morinerie
cddfb1727f
Merge pull request #5909 from betagouv/bundle-update 
Mise à jour des dépendances Ruby
 2021-02-16 17:51:21 +01:00
Pierre de La Morinerie
98be7e2076 Fix new Rubocop warnings 2021-02-16 16:31:30 +00:00
Pierre de La Morinerie
f41c1f9ff4 gems: update all dependencies 2021-02-16 16:31:30 +00:00
Pierre de La Morinerie
aae452cc67 gems: update chunky_png 
And pass it a proper IO object, to avoid "Undefined method
`set_encoding`" errors.
 2021-02-16 16:31:30 +00:00
Pierre de La Morinerie
b04afed20e gems: update graphql 2021-02-16 16:31:30 +00:00
krichtof
62a1716067
Merge pull request #5903 from betagouv/4865-multi-select-accessible 
rend accessible le multi-select
 2021-02-16 17:16:35 +01:00