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.

Sign up for GitHub

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

Jump to bottom

stop_reason_source_value should be varchar #147

Closed
gowthamrao opened this issue Jan 6, 2018 · 1 comment
Closed

stop_reason_source_value should be varchar #147

gowthamrao opened this issue Jan 6, 2018 · 1 comment
Assignees
@clairblacketer
Labels
Bug

Comments

@gowthamrao
Copy link
Member

CommonDataModel/Sql Server/OMOP CDM sql server ddl.txt

Line 626 in ced0af0

stop_reason_source_value INTEGER NULL ,

@clairblacketer stop_reason_source_value should be varchar(50). It is implemented as an integer. Would it be possible to correct this
Thank you

Gowtham
@clairblacketer clairblacketer self-assigned this Jan 8, 2018
@clairblacketer
Copy link
Contributor

@gowtham - thanks for noticing that. I will create a pull request to fix it

clairblacketer added a commit that referenced this issue Jan 8, 2018
@clairblacketer
Fixed PAYER_PLAN_PERIOD, changed STOP_REASON_SOURCE_VALUE to VARCHAR(…
878f108 
…50) from INTEGER, based on issue #147
@clairblacketer clairblacketer mentioned this issue Jan 8, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@clairblacketer clairblacketer

Labels
Bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@clairblacketer @gowthamrao