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

Use comments table for RSVP instead of custom RSVP table. #692

Merged
merged 19 commits into from
Jun 23, 2024
Merged

Conversation

mauteri
Copy link
Contributor

@mauteri mauteri commented Jun 14, 2024

Description of the Change

This change utilizes the Comments table with a comment type of gatherpress_rsvp in WordPress instead of a custom RSVP table. It also adds code for a comment tax_query, which isn't used at the moment, but will likely come in handy soon. This requires an update to GatherPress Alpha to migrated old RSVPs in custom table to Comments table, then delete the custom table.

Closes #687

How to test the Change

  • RSVP should work as it always has, but RSVPs will now be stored in Comments table.
  • A custom taxonomy _gatherpress_rsvp_status is also being used for attending, not_attending, waiting_list
  • Old RSVPs should not appear and need to be migrated with a change in GatherPress Alpha plugin.

Changelog Entry

Changed - Existing functionality

Credits

Props @mauteri, @carstingaxion

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.

Copy link

what-the-diff bot commented Jun 14, 2024

PR Summary

  • Enhancement in User Email Utilization
    Temporarily incorporated a feature to gather and use user email information across various components of our platform.

  • Improved Comment Management
    Included a new code that fetches comments and either inserts them as new or updates the existing ones based on their presence.

@carstingaxion carstingaxion linked an issue Jun 14, 2024 that may be closed by this pull request
1 task
@mauteri mauteri changed the title Started work on switching to comments table for RSVP. Use comments table for RSVP instead of custom RSVP table. Jun 22, 2024
@mauteri mauteri marked this pull request as ready for review June 22, 2024 17:50
Copy link
Collaborator

@stephenerdelyi stephenerdelyi left a comment

Choose a reason for hiding this comment

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

Just tested this by:

  • reviewing that my existing test events (with attendees previously) are no longer displaying upon checkout + build
  • signing up for a test event from the frontend
  • ensuring that user was recorded
  • added a test user under a different account
  • successfully deleted / managed users from the gutenberg backend

No unexpected results were produced as a result of these updates. Marking as approved, though we should plan to update alpha to migrate over the existing data 🎉

@mauteri mauteri merged commit 8e6d80c into main Jun 23, 2024
11 checks passed
@mauteri mauteri deleted the GP-687 branch June 23, 2024 14:16
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.

Make RSVPs a custom comment type
3 participants