Skip to content

feat(calendar) | expose current SDK options in WordPress SPW-20290 - #77

Open
amir-faghihi wants to merge 1 commit into
masterfrom
SPW-20290-wordpress-calendar-sdk-options
Open

feat(calendar) | expose current SDK options in WordPress SPW-20290#77
amir-faghihi wants to merge 1 commit into
masterfrom
SPW-20290-wordpress-calendar-sdk-options

Conversation

@amir-faghihi

Copy link
Copy Markdown

Functional Changes

Local QA sign-off

Not applicable by Codex (automated checks passed; PHPUnit and live WordPress smoke test were not available)

What is the change?

  • Expose the current calendar SDK options through both WordPress calendar shortcodes.
  • Map WordPress snake_case attributes to typed SDK camelCase options.
  • Support display pricing, hidden pricing, direct date and timeslot targets, special-event keys, and multi-day start-date-only highlighting.
  • Add shortcode documentation and focused PHP/JavaScript coverage.

Why the change is important?

  • Keeps the WordPress calendar integrations aligned with the current Showpass calendar SDK.
  • Allows WordPress embeds and modal calendars to use the same calendar targeting and presentation options as the frontend.

How to test?

  • npm run test:calendar
  • node --check plugin/js/showpass-calendar-params.js
  • php -l plugin/showpass-wordpress-plugin-shortcode.php
  • php -l tests/CalendarShortcodeTest.php
  • Exercise both [showpass_calendar_widget] and [showpass_embed_calendar] in WordPress with the documented attributes.

Database Migrations

What is the reason for the migration(s) and impact on the database?

No database migrations.

Migration Approval

Not applicable by Codex (no migrations).

Additional Notes

PHPUnit was not run because vendor/bin/phpunit is not installed in the checkout. No deploy notes required.

@amir-faghihi amir-faghihi self-assigned this Aug 26, 2026
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.

1 participant