Allow multiple SCs per issue #14
Comments
|
Hey @hidde, I've gotten this working in my fork, though there are also a lot of other changes, so probably won't open a PR from that fork. The main change was in function getSCs(principleSearchString) {
let principle = [];
for (const issue of allIssues) {
for (const sc of issue.data.sc) {
if (sc.startsWith(principleSearchString)) {
principle.push(sc);
}
}
}
return Array.from(new Set(principle));
}
const perceivable = getSCs("1.");
const operable = getSCs("2.");
const understandable = getSCs("3.");
const robust = getSCs("4.");There was also a small change needed in the report layout around the WCAG link: {% if issue.data.sc and issue.data.sc.length !== 0 %}
{% for sc in issue.data.sc %}
...
{% endfor %}
{% endif %}And lastly there was the const sanitizeNumbersAndReturnHighest = (numberWithDotsArray) => {
let number = null;
for (const numberWithDots of numberWithDotsArray) {
const numberString = numberWithDots
.split(".")
.map((numb) => (numb = Number(numb) < 10 ? "0" + numb : numb))
.join("");
number = number < Number(numberString) ? Number(numberString) : number;
}
return number;
}; |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently the
scproperty of an issue accepts one number, for instance:It would be nice if you could add an array of SCs:
The text was updated successfully, but these errors were encountered: