Skip to content

Comments

statistics: replace $.ajax with lorisFetch#10338

Open
Montekkundan wants to merge 1 commit intoaces:mainfrom
Montekkundan:montek/issue-4213-statistics-fetch
Open

statistics: replace $.ajax with lorisFetch#10338
Montekkundan wants to merge 1 commit intoaces:mainfrom
Montekkundan:montek/issue-4213-statistics-fetch

Conversation

@Montekkundan
Copy link

Summary

Replace $.ajax calls with lorisFetch in statistics.

Why

This removes jQuery AJAX usage in this module for issue #4213.

Scope

Only AJAX replacement in this module. No unrelated jQuery refactors.

Dependency

Depends on PR #10333 .

Verification

git grep '\$\.ajax' is clean for touched files.

@github-actions github-actions bot added Language: Javascript PR or issue that update Javascript code Module: statistics PR or issue related to statistics module labels Feb 5, 2026
@driusan driusan added the State: Blocked PR or issue awaiting an external event such as the merge or another PR to proceed label Feb 5, 2026
@HenriRabalais HenriRabalais self-assigned this Feb 10, 2026
@Montekkundan
Copy link
Author

reviewed this pr against the new Client guidance and did not push code changes.

  • The touched calls in this PR are HTML fragment/tab content requests (response.text() + DOM injection), not REST-style JSON data requests.
  • Per guidance, those flows remain on lorisFetch.
  • I can follow up with a separate Client migration if/when JSON endpoints are introduced for these paths.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Language: Javascript PR or issue that update Javascript code Module: statistics PR or issue related to statistics module State: Blocked PR or issue awaiting an external event such as the merge or another PR to proceed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants