Skip to:
Content

BuddyPress.org

Opened 6 months ago

Closed 3 months ago

#9126 closed defect (bug) (fixed)

Make BuddyPress preview available in WP Playground

Reported by: imath's profile imath Owned by: imath's profile imath
Milestone: 14.0.0 Priority: normal
Severity: normal Version:
Component: Build/Test Tools Keywords: needs-testing has-patch
Cc:

Description

Hi!

I just saw this notice on our plugin's WP.org plugin page:

https://i.imgur.com/W9k8r1Y.png

Can someone volunteer to perform the following checklist tasks:

Test your plugin in Playground
Fix any bugs in your plugin that prevent it from working in Playground.
Download blueprint.json
Commit your blueprint to svn.

If everything's fine, please add a patch or a PR to include the blueprint to the repo.

Thanks in advance for your help.

Change History (10)

#1 @espellcaste
6 months ago

  • Owner changed from netweb to espellcaste
  • Status changed from new to assigned

I’d be willing to do this. I was about to for one of my plugins…

This ticket was mentioned in Slack in #buddypress by espellcaste. View the logs.


5 months ago

#3 @imath
5 months ago

As requested by @espellcaste

I've just added this blueprint file: https://plugins.trac.wordpress.org/browser/buddypress/assets/blueprints/blueprint.json

https://i.imgur.com/YokxFmu.png

Then the Preview button appeared at the right of the Plugin's page Download button: here's the link of this button: https://wordpress.org/plugins/buddypress/?preview=1

Unlike what is written into this doc: https://developer.wordpress.org/plugins/wordpress-org/previews-and-blueprints/ I don't see anything into the Advanced View of the plugin page allowing me to make this preview button public.

It looks like even after having logged out and log in again, Playground doesn't keep information about the logged in user, in the screenshot below, Admin should be listed into the Last Active members.

https://i.imgur.com/GStjvN2.png

I guess the fact BuddyPress can't get information about the logged in user, it can explain the error that is displayed when posting an activity:

https://i.imgur.com/ZJdAoKE.png

Finally, by default plain permalinks are active: it's great we are now supporting these since we migrated from our Legacy URL parser to the BP Rewrites API.

Possible improvements: create a wp-navigation for Twenty Twenty-four including items for our directories.

This ticket was mentioned in Slack in #buddypress by imath. View the logs.


5 months ago

#5 @espellcaste
5 months ago

  • Owner changed from espellcaste to imath

Moving this ticket to @imath for completion since he has the necessary permissions to properly test it.

#6 @imath
4 months ago

After some debugging it looks like DB tables are not created as soon as one of the field is having the bool type.

This ticket was mentioned in PR #287 on buddypress/buddypress by imath.


4 months ago
#7

  • Keywords has-patch added

Updates BuddyPress db tables schema so that it uses tinyint fields instead of boolean ones. This is required to have WP Playground work as expected.

NB: this move is requiring a major version to be achieved.

Trac ticket: https://buddypress.trac.wordpress.org/ticket/9126

#8 @imath
4 months ago

  • Milestone changed from 12.5.0 to 14.0.0

This ticket was mentioned in Slack in #buddypress by dcavins. View the logs.


3 months ago

#10 @imath
3 months ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 13907:

Make sure BuddyPress can be previewed using WP Playground

Props espellcaste

Fixes #9126
Closes https://github.com/buddypress/buddypress/pull/287

Note: See TracTickets for help on using tickets.