Skip to content

chore: 馃嵄 add DK-EN mapping for choice values from REDCap - #176

Open
K-Beicher wants to merge 5 commits into
mainfrom
docs/data-value-translation
Open

chore: 馃嵄 add DK-EN mapping for choice values from REDCap#176
K-Beicher wants to merge 5 commits into
mainfrom
docs/data-value-translation

Conversation

@K-Beicher

Copy link
Copy Markdown
Contributor

Description

Just storing the translations of the selectable values in redcap until we need them for the data analysis extracts.

Closes #74

Needs a quick review.

Checklist

  • Ran just run-all

@K-Beicher K-Beicher self-assigned this Jul 29, 2026
@K-Beicher

Copy link
Copy Markdown
Contributor Author

Got an error on just run-all, need to investigate that before I turn this into a PR.

@K-Beicher
K-Beicher marked this pull request as ready for review July 29, 2026 10:44
@K-Beicher
K-Beicher requested a review from a team as a code owner July 29, 2026 10:44
@K-Beicher K-Beicher moved this from Todo to In review in Data development Jul 29, 2026

@lwjohnst86 lwjohnst86 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This should go under src/feasibility_data/metadata/redcap/. And also, file names need to always be lowercase

@github-project-automation github-project-automation Bot moved this from In review to In progress in Data development Jul 29, 2026
@K-Beicher
K-Beicher requested a review from lwjohnst86 July 29, 2026 11:02

@lwjohnst86 lwjohnst86 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The redcap in the name is redundant since it's in the redcap folder, so you can remove that from the name. And you could rename the last bit to dk-to-en since en is the abbreviation for English.

@lwjohnst86 lwjohnst86 moved this from In progress to In review in Data development Jul 30, 2026
@lwjohnst86 lwjohnst86 moved this from In review to In progress in Data development Jul 30, 2026

@lwjohnst86 lwjohnst86 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You moved it into metadata/redcap/, but I said src/feasibility_data/metadata/redcap/ (in the src/ folder).

@lwjohnst86 lwjohnst86 changed the title chore: upload of document translating all choice values from redcap chore: 馃嵄 add DK-EN mapping for choice values from REDCap Jul 31, 2026
@lwjohnst86 lwjohnst86 changed the title chore: 馃嵄 add DK-EN mapping for choice values from REDCap chore: 馃嵄 add DK-EN mapping for choice values from REDCap Jul 31, 2026
@K-Beicher K-Beicher moved this from In Progress to In Review in Data development Aug 5, 2026
@K-Beicher
K-Beicher requested a review from lwjohnst86 August 5, 2026 12:46
@K-Beicher K-Beicher moved this from In Review to Approved in Data development Aug 6, 2026
@K-Beicher K-Beicher moved this from Approved to In Review in Data development Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

Mapping of values in the columns between Danish and English?

2 participants