feat: add den haag file to themes and visual regression test#1353
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1353 +/- ##
=======================================
Coverage 89.51% 89.51%
=======================================
Files 19 19
Lines 1392 1392
Branches 185 185
=======================================
Hits 1246 1246
Misses 145 145
Partials 1 1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
768fc96 to
5159a8d
Compare
eslook
left a comment
There was a problem hiding this comment.
UI test laat ongewenst resultaat zien, dus ik ben dat aan het uitzoeken!
eslook
left a comment
There was a problem hiding this comment.
Ok, context, en approving:
Eerder toen Rijkshuisstijl Community een component in themes zette, gaf dit ook ongewenste UI Tests resultaten, en dat bleek te komen door CSS contamination. Dus ik zocht naar dit specifieke probleem. Maar dit probleem lijkt dus niet voor te komen bij Den Haag; de UI Tests wijziging lijkt somehow te komen door een andere PR die gemerged is, waar die specifieke wijziging (die we hier terugzien in de UI Tests) expliciet expres in gemaakt is.
Resultaat:
- ik approve de twee Den Haag PRs: ze zijn goed, en de UI Test wijziging komt somehow door die PR.
- we proberen nog uit te vinden waarom dit gebeurde, en of het weer gebeurt.
Excuses voor de vertraging!
ab1e7b1 to
334a2bd
Compare
|



Part of Community Components - Den Haag (view)
and of nl-design-system/documentatie#3308