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.

Sign up for GitHub

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

Jump to bottom

Config flag conf["viur.bone.date.localize"] to globally define localize-flag of DateBone #565

Open
phorward opened this issue Nov 28, 2022 · 1 comment · Fixed by #595
Open

Config flag conf["viur.bone.date.localize"] to globally define localize-flag of DateBone #565

phorward opened this issue Nov 28, 2022 · 1 comment · Fixed by #595
Labels
feature New feature or request idea

Comments

@phorward
Copy link
Member

This is an idea with a useful context.

Currently, localize defaults to False and must explicitly be set on a DateBone.
creationdate and changedate are never localized, but could be using such a flag.

conf["viur.bone.date.localize"] = True

would make all DateBone(date=True, time=True) localized, when not explicitly switched True or False on the bone.
@phorward phorward added feature New feature or request idea labels Nov 28, 2022
phorward added a commit to phorward/viur-core that referenced this issue Nov 28, 2022
@phorward
Globally control DateBone localization
fe8c81d 
As requested by viur-framework#565
@phorward phorward mentioned this issue Nov 28, 2022
Closed
@phorward
Copy link
Member Author

phorward commented Dec 9, 2022

VIUR meetup 2022-12-09: Why not localize=True all DateBones by default?

@phorward phorward mentioned this issue Dec 14, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature New feature or request idea
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

DateBone(localize=True) becomes default setting phorward/viur-core
1 participant
@phorward