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

Ethical: Add new ethical report uri field to frontend (disableable from the backend) #397

Open
wants to merge 1 commit into
base: next
Choose a base branch
from

Conversation

GeoffreyKarnbach
Copy link
Contributor

@GeoffreyKarnbach GeoffreyKarnbach commented Dec 11, 2024

Description

Feature

What does this PR do?

Add new ethical report URI field in the frontend step 6 ("Ethical aspects" view), the input field is only being displayed if it is not disabled through the config and if the Question "Was your research plan reviewed by an ethics committee, the Ethics Committee of your institution , or a similar body?" has been answered with yes.

A new URI and URL Validator have been introduced in a reusable manner.

Dependencies

Related Backend PR

closes GH-382

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
5.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@GeoffreyKarnbach GeoffreyKarnbach force-pushed the gk/382-ethics-add-ethical-issues-report-to-damap branch 2 times, most recently from e8cd598 to 17df067 Compare January 21, 2025 10:20
@ValentinFutterer
Copy link
Collaborator

Good work, the validators will also prove useful in the future!

Only one thing, like in the backend. Can you remove the mentions of URI and URL and rename them to ethical issues report like in the standard?

@GeoffreyKarnbach GeoffreyKarnbach force-pushed the gk/382-ethics-add-ethical-issues-report-to-damap branch 4 times, most recently from 82a4f9a to bede4e2 Compare January 29, 2025 08:50
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
6.2% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

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.

Ethics: Add ethical issues report to DAMAP
2 participants