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.

Sign up for GitHub

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

Jump to bottom

Remove jQuery AJAX from the notifications #29817

Merged
merged 2 commits into from
Mar 16, 2024
Merged

Remove jQuery AJAX from the notifications #29817

merged 2 commits into from
Mar 16, 2024

Commits on Mar 15, 2024

  1. Remove jQuery AJAX from the notifications 

    - Removed 2 jQuery AJAX calls and replaced with our fetch wrapper
- Deleted an AJAX call that wasn't attached to any element since go-gitea#24989
- Tested the notification count and notification table functionality and it works as before

Signed-off-by: Yarden Shoham <git@yardenshoham.com>
    @yardenshoham
    yardenshoham committed Mar 15, 2024
    Configuration menu
    Copy the full SHA
    71fa38b View commit details
    Browse the repository at this point in the history

  2. Make it nicer 

    Co-authored-by: silverwind <me@silverwind.io>
    @yardenshoham @silverwind
    yardenshoham and silverwind committed Mar 15, 2024
    Configuration menu
    Copy the full SHA
    c815ff0 View commit details
    Browse the repository at this point in the history