tsql_validate_field_types. should recognize "POSIXct" & "POSIXt"

PHSKC-APDE / rads

RADS Code

1 stars 1 forks source link

tsql_validate_field_types. should recognize "POSIXct" & "POSIXt" #362

Open dcolombara opened 2 months ago

dcolombara commented 2 months ago

tsql_chunk_loader() should recognize class of type "POSIXct" & "POSIXt", which is produced by class(Sys.time())

When the validate_field_types = TRUE, and the field_type is datetime in SQL, the function returns:

🛑 last_run in {ph.data} is of class 'c("POSIXct", "POSIXt")', which is not recognized by rads::tsql_validate_field_types. If you think this is a mistake, please submit a GitHub issue.

It should recognize the legitimacy of this pairing without error or warning

dcolombara commented 2 months ago

Also needs to recognize data type 'TINYINT' field_type

dcolombara commented 2 months ago

How is rads::tsql_validate_field_types different from rads::validate_yaml_data???