ecrin-github / ecrin-github/crPMT

Study countries: redo "Safety Notifications - CTIS" handling

Open
#15 0 comments 0 reactions 0 assignees View on GitHub
BE bug
Dominant language
TypeScript
Stars
0
Forks
1
PR merge metrics
No merged PRs in 30d

Description

Currently, the sharing of Safety Notifications between CTIS countries is entirely decided and handled in the FE, and country change on Study Country page + "Safety Notifications - CTIS" checkbox have been disabled on Study Country page because the feature is messy and unfinished.
Everything should be handled in the BE instead for a single source of truth, and therefore endpoints/API requests probably have to be changed too.
Note: should also probably use PATCH queries in FE instead of always PUT

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.