Use 'dttm' or 'DTTM' as the expected name for the datetime column in event data (e_data)

ThomUK / SPCreporter

Creates Metric Reports using Statistical Process Control in the NHS style

https://thomuk.github.io/SPCreporter/

Other

6 stars 2 forks source link

Use 'dttm' or 'DTTM' as the expected name for the datetime column in event data (e_data) #101

Closed francisbarton closed 6 months ago

francisbarton commented 8 months ago

This makes the e_data process more compatible with the default output from e.g. DataHoover::dh_get_data() which expects to return SQL data with ID and DTTM columns. Also it is shorter.

ThomUK commented 6 months ago

Thinking again on this, I want to leave the column name as it is: "event_date_or_datetime", because it helps with documentation, and is self-explanatory for anyone maintaining event lists manually.
It's also easy to work around with a column rename when importing automated data.