Skip to content

Hide d-portal links for ad hoc validation reports - #1006

Open
floze-the-genius wants to merge 1 commit into
IATI:developfrom
floze-the-genius:fix/hide-dportal-links-adhoc
Open

Hide d-portal links for ad hoc validation reports#1006
floze-the-genius wants to merge 1 commit into
IATI:developfrom
floze-the-genius:fix/hide-dportal-links-adhoc

Conversation

@floze-the-genius

Copy link
Copy Markdown

Fixes #999.

Summary

  • render activity identifiers as plain text in ad hoc validation reports
  • keep the existing d-portal links for published validation reports
  • cover the ad hoc behavior with a Cypress regression test

Testing

  • npm run build:development
  • npx eslint src/components/report/FeedbackGroup.vue cypress/integration/validationReport.spec.js
  • npx prettier src/components/report/FeedbackGroup.vue cypress/integration/validationReport.spec.js --check
  • npx cypress run --config-file cypress.config.local.js --spec cypress/integration/validationReport.spec.js --browser electron (11 passing)

The repository-wide npm run lint still reports the existing warning in src/utils/misc.js and an existing Prettier mismatch in package.json; neither file is changed by this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

When displaying results of an ad hoc validation, do not display d-portal links

1 participant