[FEATURE] Prendre en compte la whitelist des URLs dans la moulinette (PIX-6043) #792
+174
−13
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Une fois l'application déployée, elle sera accessible à cette adresse https://pix-lcms-review-pr792.osc-fr1.scalingo.io |
laura-bergoens
force-pushed
the
pix-6043-take-into-account-url-whitelist-in-moulinette-url
branch
from
41f5458
to
4119b25
Compare
fael-b
force-pushed
the
pix-11737-allow-admin-to-handle-urls-whitelist-for-moulinette-urls-cassés
branch
from
193c605
to
ec7b65d
Compare
laura-bergoens
force-pushed
the
pix-11737-allow-admin-to-handle-urls-whitelist-for-moulinette-urls-cassés
branch
3 times, most recently
from
d086966
to
dccdb37
Compare
laura-bergoens
force-pushed
the
pix-6043-take-into-account-url-whitelist-in-moulinette-url
branch
from
4119b25
to
02d8edf
Compare
laura-bergoens
changed the base branch from
pix-11737-allow-admin-to-handle-urls-whitelist-for-moulinette-urls-cassés
to
pix-15183-edit-whitelisted-url
laura-bergoens
force-pushed
the
pix-6043-take-into-account-url-whitelist-in-moulinette-url
branch
from
02d8edf
to
596c0fa
Compare
laura-bergoens
force-pushed
the
pix-6043-take-into-account-url-whitelist-in-moulinette-url
branch
from
596c0fa
to
36589a5
Compare
laura-bergoens
force-pushed
the
pix-15183-edit-whitelisted-url
branch
2 times, most recently
from
67f1cc6
to
0dadb6d
Compare
pix-service-auto-merge
force-pushed
the
pix-15183-edit-whitelisted-url
branch
from
0dadb6d
to
6734f7b
Compare
|
Une fois l'application déployée, elle sera accessible à cette adresse https://pix-lcms-review-pr792.osc-fr1.scalingo.io |
laura-bergoens
force-pushed
the
pix-6043-take-into-account-url-whitelist-in-moulinette-url
branch
from
36589a5
to
c4883a4
Compare
Jeremiejade
approved these changes
Nov 14, 2024
laura-bergoens
force-pushed
the
pix-6043-take-into-account-url-whitelist-in-moulinette-url
branch
from
c4883a4
to
f7d4706
Compare
laura-bergoens
added
Func Review OK
🚀 Ready to Merge
Tech Review OK
and removed
👀 Func Review Needed
👀 Tech Review Needed
labels
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🦄 Problème
Pour la moulinette, il y a des URLs qu'on sait KO de base et qu'on souhaiterait ignorer dans le fichier de résultat pour faire moins de bruit.
On a une PR pour mettre en place une interface de gestion des urls à whitelister.
Maintenant l'idée c'est de prendre en compte cette liste dans le job.
🤖 Proposition
Prendre les urls whitelistées actives
En tenir compte dans la moulinette (avec les deux types de check, exact match ou starts with)
🌈 Remarques
Dépend de la PR pour l'interface de gestion
💯 Pour tester
Tester de lancer le job avec des urls whitelistée