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.

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

Date format option #711

Closed
erroldekock opened this issue Jun 22, 2016 · 3 comments · Fixed by #2444
Closed

Date format option #711

erroldekock opened this issue Jun 22, 2016 · 3 comments · Fixed by #2444
Assignees
Milestone

Comments

@erroldekock
Copy link

In South Africa we use a date format like the UK and Europe where we display dd/mm when the year is not used. Both churchinfo and churchcrm use the normal US standard of mm/dd. This is confusing when the date is 02/06 but obvious when displaying 05/17. To fix this in my production environment for the church directory I modified the PHP to print dd/mm in the report. There are 3 or 4 places in the file where this needs to be done but it is a problem when new releases come out as I have to restore my fix or merge with the new code. Could we have a date format flag to set this?

@DawoudIO
Copy link
Contributor

DawoudIO commented Jun 22, 2016

@erroldekock join us at https://gitter.im/ChurchCRM/CRM for quicker answers

but my recommendation is to make the date format a setting and update each page that displays dates to use that setting.

@erroldekock
Copy link
Author

I am now at gitter.im. I would be happy with that enhancement as a date format setting

@crossan007 crossan007 added this to the 2.3.0 milestone Sep 11, 2016
@DawoudIO DawoudIO modified the milestones: 2.3.0, 2.4.0 Oct 23, 2016
@DawoudIO DawoudIO removed this from the 2.4.0 milestone Nov 24, 2016
@crossan007
Copy link
Contributor

These config items have been built, but we still need to find / replace all places where we have hardcoded date formats

@ghost ghost assigned crossan007 May 1, 2017
@ghost ghost added the In Review label May 1, 2017
@ghost ghost removed the In Review label May 1, 2017
@crossan007 crossan007 added this to the 2.7.4 milestone May 1, 2017
@ghost ghost added the In Review label May 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants