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

Event date block and functionality refactor #820

Merged
merged 18 commits into from
Sep 12, 2024
Merged

Event date block and functionality refactor #820

merged 18 commits into from
Sep 12, 2024

Conversation

mauteri
Copy link
Contributor

@mauteri mauteri commented Aug 27, 2024

Description of the Change

This change updates the Event Date block to look and feel more like the WordPress core date block in terms of styling. In this PR we also introduce meta data that is available in core post REST API for datetime start, end, and timezone. We also use meta data hook to update the custom event table for datetimes. This means we no longer need the datetime REST endpoint which was a bit clunky and required an additional call. That endpoint has now been removed. In addition, WP core stores powered by redux were introduced to start replacing the Broadcaster/Listener code that was custom written. This works a LOT better, and soon more things will get stores to replace Broadcaster/Listener functionality.

Closes #803 #684

How to test the Change

Changelog Entry

Added - New feature
Changed - Existing functionality
Removed - Feature
Fixed - Bug fix

Credits

Props @mauteri

Checklist:

  • I agree to follow this project's Code of Conduct.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests pass.

…ime endpoint many times on save rather than just once.
@carstingaxion
Copy link
Collaborator

@mauteri Do not miss to checkout the work at WordPress/gutenberg#60971, which seems closely related to GatherPress' needs for a close-to-core event date block.

@mauteri
Copy link
Contributor Author

mauteri commented Sep 4, 2024

@mauteri Do not miss to checkout the work at WordPress/gutenberg#60971, which seems closely related to GatherPress' needs for a close-to-core event date block.

Yup, saw this but it's still in the design phase. If this makes it into core we can definitely swap out the picker.

Copy link

Preview changes with Playground

You can preview the least recent changes for PR#820 by following one of the links below:

⚠️ Note: The preview sites are created using WordPress Playground. You can add content, edit settings, and test the themes as you would on a real site, but please note that changes are not saved between sessions.

@mauteri mauteri changed the title Started work on saving datetimes as meta; fixed bug that called datet… Event date block and functionality refactor Sep 10, 2024
@mauteri mauteri marked this pull request as ready for review September 10, 2024 16:14
@mauteri mauteri merged commit 6465f53 into main Sep 12, 2024
15 checks passed
@mauteri mauteri deleted the GP-803 branch September 12, 2024 22:27
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.

Save event start and end datetimes and timezone as meta
4 participants