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 #4518

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

[PROD-7937] In Plugin Notification system #4518

wants to merge 10 commits into from

Conversation

annrra
Copy link
Contributor

@annrra annrra commented Sep 25, 2024

Jira Issue:

https://buddyboss.atlassian.net/browse/PROD-7937

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 annrra self-assigned this Sep 25, 2024
annrra and others added 9 commits September 25, 2024 22:47
…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
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.

2 participants