[EXTERNAL] Clarify the need to implement the create_date function for birth date conversion #2744
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Clarify the need to implement the create_date function for birth date conversion
zanninso
requested changes
Oct 10, 2024
| @@ -5,6 +5,7 @@ | |||
| For this exercise, you will have to implement an **error type** for a form validator. This must validate the password and the first name. | |||
|
|
|||
| The first name must not be empty and the password must have **at least 8 characters**, and a combination of **alphabetic**, **numeric** and **none-alphanumeric** (`<`, `&`, `/` ...). | |||
| Additionally, you need to create a helper function create_date that converts a string like "2015-09-05" into a NaiveDate format. | |||
There was a problem hiding this comment.
The question for this exercise to build the error type form validator, the data will be given as input in the right format.
they can use NaiveDate::parse_from_str(date_str, "%Y-%m-%d") if needed for test so no need to create a function to converts the date
| @@ -15,7 +16,7 @@ Examples: | |||
| Create a structure named `Form` that will have the following fields: | |||
| - `first_name`: `String` | |||
| - `last_name`: `String` | |||
| - `birth`: `NaiveDate` that will convert a string `"2015-09-05"` to a date of that format. | |||
| - `birth`: `NaiveDate` (converted from a string using the create_date function). | |||
There was a problem hiding this comment.
The description here have no meaning since we have a data type NaiveDate.
Please remove the description from here then i can approve your PR
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why?
This update ensures that all the requirements of this question are clear, avoiding confusion and time waste.