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

Add common Dutch expressions for day in last or next week, fixes #44 #45

Merged
merged 1 commit into from
Dec 2, 2023

Conversation

iamreinder
Copy link
Collaborator

I implemented common Dutch expressions for a day in the last week, for example "vorige week woensdag" which translates to "last week on wednesday". Or a day in the next week, like "volgende week zondag" which translates to "next week on sunday." This is commonly used and is different from "next sunday." If I say today (on a friday) that I will see you "volgende week zondag" I mean the sunday after next sunday. Although it is less commonly used, I also added "komende week" which means the same as "volgende week."

This fixes the issue described in #44.

@iamreinder iamreinder requested review from olebedev and RexSkz December 1, 2023 19:09
Copy link
Collaborator

@RexSkz RexSkz left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Owner

@olebedev olebedev left a comment

Choose a reason for hiding this comment

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

LGTM 🚀 (I have no idea what I am approving, really :D)

@iamreinder
Copy link
Collaborator Author

Hehe, I can imagine. I bet I would be in the same position when a feature like this was pull requested for the Chinese language 😅

Let's just say I implemented another way to describe a day in the past or future for the Dutch language.

@iamreinder iamreinder merged commit 3292cb6 into olebedev:master Dec 2, 2023
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.

3 participants