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

Cleanup global Notifications code #4431

Merged
merged 3 commits into from
Mar 7, 2025
Merged

Cleanup global Notifications code #4431

merged 3 commits into from
Mar 7, 2025

Conversation

theskumar
Copy link
Member

@theskumar theskumar commented Mar 6, 2025

  • Remove saas code for styling the global notification feed.
  • Update/fix the default view to show last 30 days activities.
  • Add support for quering the last 3 month activities
  • Remove unused 'notifications_dropdown.html' html and python code
  • update the admin bar to use the adminbar component.

- Remove saas code for styling the global notification feed.
- Update/fix the default view to show last 30 days activities.
- Add support for quering the last 3 month activities
- Remove unused 'notifications_dropdown.html' html and python code
@theskumar theskumar self-assigned this Mar 6, 2025
@frjo frjo merged commit 2f31a47 into main Mar 7, 2025
7 checks passed
@frjo frjo added Type: Maintenance Type: Patch Mini change, used in release drafter labels Mar 7, 2025
@theskumar theskumar deleted the cleanup/sass-notifications branch March 11, 2025 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Maintenance Type: Patch Mini change, used in release drafter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants