Commit graph

12057 commits

Author SHA1 Message Date
Pierre de La Morinerie
873cea8c86
Merge pull request #5972 from betagouv/rspec-5
Mise à jour de rspec 3 vers rspec 5
2021-03-09 15:58:37 +01:00
Pierre de La Morinerie
4c3fcfeec2 specs: upgrade to rspec 5
Rspec 5 is compatible with Rails 6.1.
2021-03-09 15:51:19 +01:00
Pierre de La Morinerie
5bf499baa0 specs: remove rspec_junit_formatter
It was unsed by CircleCI to generate test reports in the JUnit XML
format, but:

- We now use Github Actions, which has its own reporting system,
- It prevents us to upgrade to rspec > 3.
2021-03-09 15:51:19 +01:00
Paul Chavard
6b52390324
Merge pull request #5934 from tchak/blank-default-groupe-instructeur
Le sélecteur du groupe instructeur est vide par défaut
2021-03-09 14:54:43 +01:00
Paul Chavard
00d7c3186a fix indentation 2021-03-09 14:47:23 +01:00
Paul Chavard
70500e3d56 Groupe instructeur selector default blank 2021-03-09 14:47:23 +01:00
Paul Chavard
9c3e72c84a Use revision instead of groupe instructeur as dossier procedure link 2021-03-09 14:47:23 +01:00
Pierre de La Morinerie
1e10298859
Merge pull request #5967 from betagouv/dependabot/npm_and_yarn/elliptic-6.5.4
build(deps): bump elliptic from 6.5.3 to 6.5.4
2021-03-09 14:27:35 +01:00
dependabot[bot]
f87efe9670
build(deps): bump elliptic from 6.5.3 to 6.5.4
Bumps [elliptic](https://github.com/indutny/elliptic) from 6.5.3 to 6.5.4.
- [Release notes](https://github.com/indutny/elliptic/releases)
- [Commits](https://github.com/indutny/elliptic/compare/v6.5.3...v6.5.4)

Signed-off-by: dependabot[bot] <support@github.com>
2021-03-09 13:20:03 +00:00
Paul Chavard
c2fed3f952
Merge pull request #5969 from tchak/remove-circle-ci
Remove circle ci config
2021-03-09 14:03:13 +01:00
Paul Chavard
d04d9ea5d8 Remove circle ci config 2021-03-09 13:50:45 +01:00
Kara Diaby
8157fe8c5c
Merge pull request #5971 from betagouv/revert-5965-claimant-type-to-avis-table
Revert "Claimant type to avis table"
2021-03-09 13:49:07 +01:00
Kara Diaby
f4fd220d43
Revert "Claimant type to avis table" 2021-03-09 13:40:10 +01:00
Pierre de La Morinerie
0c5ad449c5
Merge pull request #5968 from tchak/fix-carto-ie-again
Correction (à nouveau) du chargement des cartes dans Internet Explorer 11 (#5968)
2021-03-09 11:46:54 +01:00
Paul Chavard
1644c9d44b Another attempt at fixing IE 2021-03-09 11:29:20 +01:00
Kara Diaby
d124512336
Merge pull request #5965 from betagouv/claimant-type-to-avis-table
Claimant type to avis table
2021-03-09 11:04:15 +01:00
kara Diaby
0870ebfd90 After party backfill claimant type on avis table 2021-03-09 10:15:19 +01:00
kara Diaby
0211d1f105 modify create_avis_concern 2021-03-07 22:47:53 +01:00
kara Diaby
af4a27f4e0 add claimant type on avis table 2021-03-07 22:47:24 +01:00
krichtof
bdcce19841
Merge pull request #5963 from tchak/fix-association-serilizer
Make more association fields nullable
2021-03-05 10:26:22 +01:00
Paul Chavard
82fca8be0b Make more association fields nullable 2021-03-05 10:13:22 +01:00
Paul Chavard
5525fc7be8
Merge pull request #5962 from betagouv/fix-invalid-proc
fix limit date for notice validation
2021-03-05 09:46:32 +01:00
Christophe Robillard
6169399a6b fix limit date for notice validation 2021-03-05 08:34:39 +01:00
Pierre de La Morinerie
3969556e31
Merge pull request #5960 from betagouv/fix-warnings
Suppression de nouveaux messages d'avertissements pendant les tests et lors de l'exécution de commandes `bin/rails` (#5960)
2021-03-04 23:07:33 +01:00
Pierre de La Morinerie
f9401cfbad spec: better split spec_helper and rails_helper
This fixes the

> DEPRECATION WARNING: Initialization autoloaded the constants ActionText::ContentHelper and ActionText::TagHelper.

error appearing when running specs.

It also gets us closer from a newly-generated rails config.
2021-03-04 22:59:39 +01:00
Pierre de La Morinerie
71c4ce67b2 gemfile: don't require i18n-tasks by default
i18n-tasks is only used as a standalone command-line tool. It doesn't
need to be included in the app.

It also prevent a parser warning from appearing when running any rails
command:

> warning: parser/current is loading parser/ruby27, which recognizes
> warning: 2.7.2-compliant syntax, but you are running 2.7.1.
> warning: please see https://github.com/whitequark/parser#compatibility-with-ruby-mri.

Now this (legitimate) warning only appears when running the i18n-tasks
command.
2021-03-04 22:52:19 +01:00
Pierre de La Morinerie
b0735aafe0 config: fix DynamicSmtpSettingsInterceptor warning again
This warning re-appeared when running mailer tests:

```
DISABLE_SPRING=1 bin/rspec spec/mailers/administration_mailer_spec.rb
```

It is now fixed properly, in a way recommanded by the documentation.
2021-03-04 22:52:19 +01:00
krichtof
abb8b04208
Merge pull request #5961 from tchak/fix-carto
fix mapbox-gl transpilation
2021-03-04 19:46:40 +01:00
Paul Chavard
d4a34edab5 fix mapbox-gl transpilation 2021-03-04 19:04:39 +01:00
Pierre de La Morinerie
c8d775af29
Merge pull request #5958 from tchak/make_association_date_creation_nullable
GraphQL : corrige la date de création des associations
2021-03-04 16:41:24 +01:00
Paul Chavard
47e232f439 Some associations have no date_creation 2021-03-04 16:28:20 +01:00
krichtof
2a5d7cf696
Merge pull request #5957 from betagouv/fix-invalid-proc
no validation for old procedures
2021-03-04 16:26:31 +01:00
Christophe Robillard
7f496c43e5 no validation for old procedures 2021-03-04 16:13:19 +01:00
Pierre de La Morinerie
ae8df82dbc
Merge pull request #5953 from betagouv/fix-blob-hooks
ActiveStorage : correction de la manière dont sont déclenchés les jobs externes (anti-virus, filigranage) (#5953)
2021-03-04 14:38:03 +01:00
Pierre de La Morinerie
2f948f7e46 active_storage: fix blob update hooks
For some reason on Rails 6.1 the `after_update_commit` hook is properly
registered – but disappears from the record later, and in the end is
never run.

Fix it by using the general `after_commit` hook instead.
2021-03-04 14:30:38 +01:00
Paul Chavard
21980a37cc
Merge pull request #5947 from tchak/procedure_reset_revision
When procedure is reset delete only draft revision dossiers
2021-03-04 14:15:58 +01:00
Paul Chavard
f31096b33e When procedure is reset delete only draft revision dossiers 2021-03-04 13:10:58 +01:00
Pierre de La Morinerie
bced5fbd51
Merge pull request #5951 from tchak/make_street_address_nullable
Some addresses have no street_address
2021-03-04 12:59:39 +01:00
Paul Chavard
79606e8696 Some addresses have no street_address 2021-03-04 12:52:01 +01:00
Paul Chavard
f2e9b04f99
Merge pull request #5946 from tchak/routage-with-api
Implementer le feature flag pour permettre le routage via l'API
2021-03-04 11:20:40 +01:00
Paul Chavard
600f49a0ff Hide groupe instructeur selector when routage is done via API 2021-03-04 11:06:04 +01:00
Paul Chavard
1c811083c0 Cleanup feature flags usage 2021-03-04 11:06:04 +01:00
Paul Chavard
46377fb7d9
Merge pull request #5920 from betagouv/fix-tdc-query
procedure: fix query for finding champs without stable_id
2021-03-04 11:05:35 +01:00
Pierre de La Morinerie
9537342c06 procedure: fix query for finding champs without stable_id
It would break on ActiveRecord 6.1.
2021-03-04 09:54:22 +00:00
Paul Chavard
7c44662f23
Merge pull request #5943 from tchak/groupe_instructeur_order
Groupe instructeur order
2021-03-04 10:53:25 +01:00
Paul Chavard
5d4f025be0 Rename for consistency with defaut_groupe_instructeur 2021-03-04 10:45:02 +01:00
Paul Chavard
89ec3923f2 Defaut groupe instructeur ordered by label 2021-03-04 10:35:44 +01:00
Paul Chavard
2ea94f9eb6
Merge pull request #5944 from tchak/handle-external-data-api-errors
Add proper external data API errors handling
2021-03-04 10:35:04 +01:00
Paul Chavard
9f8c55969d Add proper external data API errors handling 2021-03-04 10:16:06 +01:00
Paul Chavard
e5a9e90304
Merge pull request #5945 from tchak/fix_send_to_instructeurs
Fix JSON.parse crashing on empty values
2021-03-04 10:11:03 +01:00