Skip to content

Ready for review#18

Open
Solomonlol wants to merge 10 commits into
the-csharp-academy:masterfrom
Solomonlol:master
Open

Ready for review#18
Solomonlol wants to merge 10 commits into
the-csharp-academy:masterfrom
Solomonlol:master

Conversation

@Solomonlol

Copy link
Copy Markdown

No description provided.

@TheCSharpAcademy TheCSharpAcademy self-assigned this Jul 23, 2026

@TheCSharpAcademy TheCSharpAcademy left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Solomonlol Thanks for submitting, there's an issue with the way you're saving dates. Because I'm in Australia I'm getting this error:

Image

Dates are saved using:

date.ToString()

That output depends on the computer’s current culture. When records are read, the application expects one exact format:

DateTime.ParseExact(
    reader.GetString(3),
    "dd.MM.yyyy H:mm:ss",
    CultureInfo.InvariantCulture)

These formats are not guaranteed to match. For example, a machine using an English-US culture may save a value containing slashes and AM/PM, which would then cause ParseExact to throw when the records are displayed.

Store and read the date using the same explicit format.

Let me know

@Solomonlol

Copy link
Copy Markdown
Author

I assume the problem has been resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants