Project

General

Profile

Actions

Bug #13352

closed

jQuery update for bp-nelo

Added by Boone Gorges about 4 years ago. Updated over 3 years ago.

Status:
Resolved
Priority name:
Normal
Assignee:
Category name:
-
Target version:
Start date:
2020-09-21
Due date:
% Done:

0%

Estimated time:
Deployment actions:

Description

In #13296 I enqueued jquery-migrate on bp-nelo, so that we didn't experience unexpected breakage after the WP 5.5 upgrade.

In advance of WP 5.6, where the default jQuery will be updated to the latest version, we should preemptively update bp-nelo's jQuery to the latest. Let's use this ticket to track the necessary changes.


Related issues

Related to CUNY Academic Commons - Bug #13296: Enqueue jquery-migrateResolvedBoone Gorges2020-09-07

Actions
Related to CUNY Academic Commons - Bug #13494: Brief Descriptor and About You Profile Fields do not display on mobile devicesResolvedBoone Gorges2020-10-23

Actions
Related to CUNY Academic Commons - Feature #13789: Update WordPress to 5.6.xResolvedBoone Gorges2021-01-07

Actions
Related to CUNY Academic Commons - Support #14129: can only see some invites sentResolvedRaymond Hoh2021-03-06

Actions
Related to CUNY Academic Commons - Bug #14716: My Commons activity blankResolvedRaymond Hoh2021-08-17

Actions
Actions #1

Updated by Boone Gorges about 4 years ago

  • Related to Bug #13296: Enqueue jquery-migrate added
Actions #2

Updated by Boone Gorges about 4 years ago

  • Related to Bug #13494: Brief Descriptor and About You Profile Fields do not display on mobile devices added
Actions #3

Updated by Boone Gorges almost 4 years ago

  • Target version changed from 1.18.0 to 1.19.0

I spent a good deal of time trying to sort out bp-nelo and the plugins that we run on the main site. The rabbit hole is deep here: cac-advanced-profiles, for example, uses a library (jquery-waypoints) that has since been updated for the latest jQuery, but this update included major changes to the API. There are other cases like this. I'm going to keep working on it during the 1.19 cycle. In the meantime, if we see a WP update that changes the default version of jQuery, we'll ensure that the migrate tools are available.

Actions #4

Updated by Boone Gorges almost 4 years ago

Actions #5

Updated by Boone Gorges almost 4 years ago

  • Target version changed from 1.19.0 to 1.18.5

Let's get to a workable solution for the WP 5.6 upgrade, scheduled for late February.

Actions #6

Updated by Boone Gorges over 3 years ago

  • Target version changed from 1.18.5 to 1.18.7

Bumping this to March for further consideration.

I've finished much of the necessary work on this (see https://github.com/cuny-academic-commons/cac/commits/wp56) but I need to push this out at the same time as the broader #13343.

Actions #7

Updated by Boone Gorges over 3 years ago

  • Status changed from New to Staged for Production Release
Actions #8

Updated by Boone Gorges over 3 years ago

  • Status changed from Staged for Production Release to Resolved
Actions #9

Updated by Raymond Hoh over 3 years ago

In the following commit, we switched from the jQuery Cookie plugin to js-cookie. This broke the bp_get_cookies() function call used in bp-nelo. See #14129.

I've just updated the bp_get_cookies() function so it uses the js-cookie logic and applied the fix to production: https://github.com/cuny-academic-commons/cac/commit/07eabf86ece2dc996038a57e2fd2641b5305df83

Actions #10

Updated by Raymond Hoh over 3 years ago

Actions #11

Updated by Raymond Hoh over 3 years ago

Added a couple of other jQuery deprecated fixes for bp-nelo: https://github.com/cuny-academic-commons/cac/compare/07eabf8...df9d9f2

Actions #12

Updated by Boone Gorges over 3 years ago

Thanks, Ray, and sorry for botching that part of the migration :-/

Actions #13

Updated by Raymond Hoh about 3 years ago

  • Related to Bug #14716: My Commons activity blank added
Actions

Also available in: Atom PDF