search

data-to-insight / csc-validator-be-cin

1 stars 1 forks source link

Update affected fields on 8675Q #439

Closed SLornieCYC closed 1 year ago

SLornieCYC commented 1 year ago

Closes #438

tab1tha commented 1 year ago

Thank you @SLornieCYC.

To get it picked it in the list of issues, we'll need to add the column name to the push_type_1 statement here https://github.com/data-to-insight/CIN-validator/blob/2a10b8950f4bb3fdc41a35fadc1bdc4281eb0e40/cin_validator/rules/cin2022_23/rule_8675Q.py#L75 and here https://github.com/data-to-insight/CIN-validator/blob/2a10b8950f4bb3fdc41a35fadc1bdc4281eb0e40/cin_validator/rules/cin2022_23/rule_8675Q.py#L146

Should I do it or do you prefer to?

SLornieCYC commented 1 year ago

Thanks @tab1tha now added.