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

fix: use correct representation of date and time when exporting XML/GML #1219

Merged

Conversation

emanuelaepure10
Copy link
Contributor

@emanuelaepure10 emanuelaepure10 commented Jun 28, 2024

Resolves issue introduced in 5.2.0 release.
Conversion to XmlDate and XmlDateTime was done but the result not used.

Copy link
Member

@stempler stempler left a comment

Choose a reason for hiding this comment

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

Would be great to have a bit more meaningful message for the changelog.

@emanuelaepure10 emanuelaepure10 changed the title fix: XmlDate and XmlDatetime converted value fix: return the correct value after conversion for XmlDate and XmlDateTime types Jul 1, 2024
@stempler
Copy link
Member

stempler commented Jul 2, 2024

@emanuelaepure10 Thanks a lot for updating the commit message. I think that's already a lot better, but I think we can still improve it in a way that it is more meaningful to a user, e.g. make it clear what functionality is impacted.
Technical details can be mentioned in the message body.

What do you think about this for example?

fix: use correct representation of date and time when exporting XML/GML

Resolves issue introduced in 5.2.0 release.
Conversion to XmlDate and XmlDateTime was done but the result not used.

Resolves issue introduced in 5.2.0 release.
Conversion to XmlDate and XmlDateTime was done but the result not used.
@emanuelaepure10 emanuelaepure10 changed the title fix: return the correct value after conversion for XmlDate and XmlDateTime types fix: use correct representation of date and time when exporting XML/GML Jul 3, 2024
@emanuelaepure10
Copy link
Contributor Author

emanuelaepure10 commented Jul 3, 2024

@stempler I have updated the comment.

@stempler stempler added the challenged For PRs to indicate that the implementation has been challenged label Jul 3, 2024
@stempler stempler merged commit 305b7ea into halestudio:master Jul 3, 2024
8 of 9 checks passed
@stempler stempler deleted the fix/date-time-conversion branch July 3, 2024 12:07
Copy link

we-release bot commented Jul 4, 2024

🎉 This PR is included in version 5.2.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@we-release we-release bot added the released label Jul 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
challenged For PRs to indicate that the implementation has been challenged released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants