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: compatible with the events calendar #1353

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

sapayth
Copy link
Member

@sapayth sapayth commented May 2, 2023

WPUF's event calendar post form wasn't working for version 6.x.x

new hook introduced
Filter hook: wpuf_tib_event_meta
Description: Opportunity to change The Event Calendar metadata just before WPUF is saving it to DB
@param array $event_data The event metadata
@param int $post_id The post id, in other words, The Event

Action hook: wpuf_tib_after_saving_event_meta
Description: Hook fired just after WPUF saved The Event Calender metadata to the DB
@param int $post_id The post id, in other words, The Event

Some test scenarios

  1. Check backward compatibility for The events calender version less than 6
  2. Test the hooks mentioned above. Here is an example of how to use hooks in WordPress.
  3. Check without the WPUF Pro version

related PR in pro #512
fix #505

@sapayth sapayth changed the title Fix/compatible with the events calendar fix: compatible with the events calendar May 2, 2023
@Rat01047 Rat01047 added needs: developer feedback This PR needs developer feedback or code changes and removed needs: testing labels Jul 12, 2023
@Rat01047 Rat01047 added the bug label Feb 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug needs: developer feedback This PR needs developer feedback or code changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants