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

[PROD-7937] In Plugin Notification system #4579

Open
wants to merge 84 commits into
base: release
Choose a base branch
from
Open

Conversation

chetansatasiya
Copy link
Collaborator

Jira Issue:

General Note

Keep all conversations related to this PR in the associated Jira issue(s). Do NOT add comment on this PR or edit this PR’s description.

Notes to Developer

  • Ensure the IDs (i.e. PROD-1) of all associated Jira issues are reference in this PR’s title
  • Ensure that you have achieved the Definition of Done before submitting for review
  • When this PR is ready for review, move the associate Jira issue(s) to “Needs Review” (or “Code Review” for Dev Tasks)

Notes to Reviewer

  • Ensure that the Definition of Done have been achieved before approving a PR
  • When this PR is approved, move the associated Jira issue(s) to “Needs QA” (or “Approved” for Dev Tasks)

annrra and others added 30 commits September 25, 2024 22:37
…ogic to switch between tabs and dismiss notices
- Add class for in plugin notification, Also load that class as autoload
- Called css and js from class
- Removed required_once, instead of that use `add_action( 'in_admin_header', array( $this, 'bb_admin_notification_header' ), 0 );` to call html in admin header
- Remove unwanted changes
- uncommented code to direct access class file
- Update doc block comment
* release: (58 commits)
Update: Grunt
Update: Grunt string-replace
Update: Version name update
Added change log
grunt
grunt fix
PROD-7989 - pulled api changes from repo
PROD-8055 - updated suffix to be more precise
PROD-8055 - Added “rapydapps.cloud” suffix to URLs considered as staging sites for Rapyd Customer
PROD-8044 - Fix - Release notes not display correctly if downgrade version
PROD-7681: Fixed profile type visibility on activity reaction reacted user list
PROD-7938 - Optimise memory
PROD-7938 - phpcbf
PROD-7938 - html formatting
PROD-7938 - Update doc block comment
PROD-7938 - Display notice again once user logout. So need to delete transient when admin logged out
PROD-7938 - Move ajax action into bp-core-admin-actions.php file. - Code refator
PROD-7938 - Update transient for 24h
PROD-7938 - Update icon
PROD-7998 - Refactor: Modify the variables and add comments.
...

# Conflicts:
#	src/bp-core/admin/bp-core-admin-actions.php
- Display notification based on fetched notification from servers
- Update action name - admin_init to bp_admin_init
- Display notification dynamically
- Also update code to dismiss notice
…uld not display

- Code refactor
- Update message if not any notifications
* release:
Update: Npm run update
Update: Grunt string-replace
Update: Version name update
Added release note
PROD-7990 - pulled api changes from repo
PROD-7420: Fix GIF disappearance issue in forum, topic, and reply actions
PROD-7922: Fix social to LearnDash group sync courses tab in group creation steps
PROD-7922: Fix social to LearnDash group sync issue on frontend group creation
PROD-7733 - Update doc block comment - Remove loggde-in condition because already checking where filter applied
PROD-7733: Removed additional token check and transient
PROD-7733: Fix issue with adding event to iCalendar on private site
- Added since tag
- Add dit at the end of the line for doc block comment
* release: (65 commits)
Update: Grunt
Update: Grunt string-replace
Update: Version name update
Added change log
grunt
PROD-7997: Fixed lastname and nickname
[PROD-7757] geunt makepot
PROD-7997 - phpcbf fixes
PROD-7997 - code refactor
PROD-7757 translation
[ARCH-131] Code cleanup
[ARCH-131] Show suspend/unsuspend button tooltip over sidearea navigation
PROD-7997: Update xprofile field visibility migration logic
PROD-7997: Set xprofile field visibility when user update profile
[PROD-7757] Update activity notice for new profile avatar
PROD-7997: Fixed xprofile field visibility when user registered from backend
PROD-7997: Run xProfile field visibility migration on plugin update
PROD-7997: Fix xProfile field visibility migration
Revert "PROD-7997: Fix member search filter results in the member directory for non-admin users"
[ARCH-131] PROD-7948 - Add "suspend_conditions (user_suspended, hide_parent, hide_sitewide)" index for bp_suspend table - Add "user_id (user_id)" index for bp_suspend_details table
...
@chetansatasiya chetansatasiya changed the title Prod 7937 [PROD-7937] In Plugin Notification system Dec 12, 2024
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.

4 participants