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

[candidate_parameters] Date format does not match dates specified in config #6678

Closed
CamilleBeau opened this issue Jun 8, 2020 · 7 comments · Fixed by #6719
Closed

[candidate_parameters] Date format does not match dates specified in config #6678

CamilleBeau opened this issue Jun 8, 2020 · 7 comments · Fixed by #6719
Assignees
Labels
Category: Bug PR or issue that aims to report or fix a bug

Comments

@CamilleBeau
Copy link
Contributor

Describe the bug
Altering the DOB format or the DOD format fields in the configuration module do not change how the dates are displayed / stored on the front-end or back-end for this module. The TestPlan for this module specifies that it should :

  1. Check that the date stored matches the DOB Format specified in the configuration module.
  1. Check that the date stored matches the DOD Format specified in the configuration module.

To Reproduce
Steps to reproduce the behavior (attach screenshots if applicable):

  1. From admin user, modify the values for DOD format and DOB format in the config module.
  2. Navigate to a candidate's profile on the candidate parameters module and see that the dates remain the same.
  3. Get the information in the back-end from the candidate table and see that the dates are stored in the same way.

What did you expect to happen?
The format of the dates to change in the candidate_parameters module.

@CamilleBeau CamilleBeau added Category: Bug PR or issue that aims to report or fix a bug 23.0.0-testing labels Jun 8, 2020
@johnsaigle

This comment has been minimized.

@driusan
Copy link
Collaborator

driusan commented Jun 9, 2020

It's unrelated to that, the config for DoB format was added in #2617

@ridz1208 ridz1208 added the Critical to release PR or issue is key for the release to which it has been assigned label Jun 9, 2020
@jesscall
Copy link
Contributor

jesscall commented Jun 9, 2020

@CamilleBeau The display on front-end isn't supposed to change or anything.

Expected behaviour when DOB / DOD format is 'Ym'

  • If user enters 2020-05-24, what is actually saved is 2020-05-01. The day of the month gets set to 01. You should notice the change on refresh

Can you re-test and confirm if that's not the case?

@CamilleBeau
Copy link
Contributor Author

@jesscall Retested. After saving the configuration settings to Ym format for DoB and DoD, the date format remained the same after refreshing and navigating away in the candidate parameter page, as well as in a query to the database.

These examples show DoB but the same was true for DoD.

image

image

image

@jesscall
Copy link
Contributor

jesscall commented Jun 9, 2020

@CamilleBeau querying DB? are you using raisinbread or the CCNA project? if CCNA: likely candidate_parameters override on CCNA is interfering and accounts for the unexpected behaviour. I quickly checked on the test vm and functionality seems to be as expected. Maybe you can confirm? NVM

@jesscall
Copy link
Contributor

jesscall commented Jun 9, 2020

@CamilleBeau also side note: the date format from your image Ym indicates using month names (Jan, Feb, etc). YM would be numeric month format.

Not sure if that's the reason for discrepancy in behaviour

Edit: Checked if changing the date format to use Ym instead of YM was the cause of the issue you reported, and it is most definitely the case. Will make the fix. Thanks for the screenshots!

@ridz1208
Copy link
Collaborator

ridz1208 commented Jun 9, 2020

Removing critical to release as this has always been the case.

@ridz1208 ridz1208 removed the Critical to release PR or issue is key for the release to which it has been assigned label Jun 9, 2020
driusan pushed a commit that referenced this issue Aug 11, 2020
Add "date_format" config format type for DoB and DoD, which configures whether day of month is included in date.

    Resolves #6678
AlexandraLivadas pushed a commit to AlexandraLivadas/Loris that referenced this issue Jun 15, 2021
)

Add "date_format" config format type for DoB and DoD, which configures whether day of month is included in date.

    Resolves aces#6678
AlexandraLivadas pushed a commit to AlexandraLivadas/Loris that referenced this issue Jun 29, 2021
)

Add "date_format" config format type for DoB and DoD, which configures whether day of month is included in date.

    Resolves aces#6678
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Bug PR or issue that aims to report or fix a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants