Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nett-1.4.3b er klar for ks #205

Merged
merged 7 commits into from
Nov 27, 2023
Merged

nett-1.4.3b er klar for ks #205

merged 7 commits into from
Nov 27, 2023

Conversation

adilsofficial
Copy link
Contributor

@adilsofficial adilsofficial commented Sep 11, 2023

Denne testregelen har kun vært i bruk en gang, i SAS tilsynet. Vi vurderer at kontrast på bilde av tekst ikke er kjerne dekning. Vi forslår derfor å videreføre testregelen uten oppdatering.

  • Vi endrer kun filnavn dvs. nett-1.4.3b.json, krav til samsvar og testregel-ID dvs. nett-1.4.3b.
  • Vi lager heller ikke testregel for app med samme begrunnelse.

@janBeniaminKwiek
Copy link
Contributor

Vet ikke om dette er riktig sted å spør dette, hva er begrunnelsen for at dette er utenfor kjernedekning?

Copy link
Contributor

@janBeniaminKwiek janBeniaminKwiek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok når skrivefeil er rettet

@geirsf
Copy link
Contributor

geirsf commented Sep 25, 2023

Vet ikke om dette er riktig sted å spør dette, hva er begrunnelsen for at dette er utenfor kjernedekning?

Er dette begrunna et sted? Altså hvorfor vi mener at bilder av tekst er utenfor kjernedekning? Og er forholdet til 1.4.5 Bilder av tekst vurdert?

@Sivzilla82
Copy link
Contributor

Vet ikke om dette er riktig sted å spør dette, hva er begrunnelsen for at dette er utenfor kjernedekning?

Denne testregelen har kun vært i bruk en gang, i SAS tilsynet. Vi vurderer at kontrast på bilde av tekst ikke er kjernedekning. Vi forslår derfor å videreføre testregelen uten oppdatering.

Co-authored-by: geirsf <geirsindre.fossoy@digdir.no>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants