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

Include 12.0 deprecated functions for new installs #183

Closed
wants to merge 2 commits into from

Conversation

imath
Copy link
Member

@imath imath commented Oct 31, 2023

  • Include 12.0 deprecated functions for new installs.
  • Add an Admin Notification about 12.0 for new installs.

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


This Pull Request is for code review only. Please keep all other discussion in the BuddyPress Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the WordPress Core Handbook for more details.

@imath
Copy link
Member Author

imath commented Nov 4, 2023

Hi @dcavins

Could you have a look at the notification text and eventually fix some of my english mistakes and/or improve it ? Thanks in advance.

I've just tested the PR and I confirm 12.0 deprecated functions are loaded even if it's a new BuddyPress install. Here's a screenshot of the Admin notification.

admin-notification-4-new-installs

I have mixed feelings about this Admin Notification because for new installs the "Hello BuddyPress" dialog's popping to inform about what's new. As a result, unless Administrators don't read this dialog, adding an Admin Notification might make some of them feel like "BuddyPress really wants me to install the BP Classic plugin".

@dcavins what do you think? Are we repeating ourselves with this Admin Notification ?

@dcavins
Copy link
Contributor

dcavins commented Nov 4, 2023

Hmm, no I don't think we're repeating ourselves, but we might re-word the notice a little to be more "if you're site is borked, try Classic before freaking out" Maybe:
BuddyPress 12 introduces major core changes, overhauling the way that BuddyPress builds and parses URLs. If you find that your site is not working correctly with the new version, try installing the new BP Classic Add-on that adds backward compatibility for plugins and themes that have not yet been updated to work with BP 12. Thanks!

@imath
Copy link
Member Author

imath commented Nov 4, 2023

Awesome, thanks for your reply, I'll update the notice this way 👍

@imath imath deleted the patch/9015 branch November 5, 2023 07:38
dcavins pushed a commit to dcavins/buddypress-wp-svn that referenced this pull request Dec 12, 2023
- Load 12.0 deprecated functions even in new installs.
- Add an Admin Notification to inform about BP Classic: the BP backwards compatibility  Add-on.

Props emaralive, dcavins

Fixes #9015
Closes buddypress/buddypress#183



git-svn-id: http://buddypress.svn.wordpress.org/trunk@13632 cdf35c40-ae34-48e0-9cc9-0c9da1808c22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants