Skip to content

document_repository: replace $.ajax with lorisFetch#10337

Open
Montekkundan wants to merge 2 commits intoaces:mainfrom
Montekkundan:montek/issue-4213-document-repository-fetch
Open

document_repository: replace $.ajax with lorisFetch#10337
Montekkundan wants to merge 2 commits intoaces:mainfrom
Montekkundan:montek/issue-4213-document-repository-fetch

Conversation

@Montekkundan
Copy link

Summary

Replace $.ajax calls with lorisFetch in document_repository.

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: document_repository PR or issue related to documen_repository 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

Updated

  • Added module client: modules/document_repository/jsx/DocumentRepositoryClient.js
  • Updated modules/document_repository/js/document_repository_helper.js:
    • editModal() now uses module client for getFileData.php with Query-based request flow.

Notes

Copy link
Collaborator

@HenriRabalais HenriRabalais left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Client implementation looks good!

@HenriRabalais HenriRabalais removed their assignment Feb 13, 2026
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: document_repository PR or issue related to documen_repository 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

Comments