adamsaghy commented on PR #6195: URL: https://github.com/apache/fineract/pull/6195#issuecomment-5177534645
> > I see no benefit of adding this field. If you want to have a sorted list, feel free to sort by created date time / and return that... > > The benefit isn't sorting — it's the displayed value itself. `submittedOnDate` is date-only, so any UI that reads it always renders a time of `00:00:00`, even though the actual creation timestamp exists in `acc_gl_journal_entry.created_on_utc`. Without `submittedOnDateTime`, there's no way for a client to show _when_ a journal entry was actually created, only _what day_. > > `submittedOnDate` is left untouched for backward compatibility, so existing consumers (including anything sorting by it) are unaffected — this is purely additive for clients that want the real timestamp. Disclose the created date time then, but no need to introduce a new field when there is one with the required data at hand. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
