Skip to content

Comments

configuration: replace $.ajax with lorisFetch#10336

Open
Montekkundan wants to merge 3 commits intoaces:mainfrom
Montekkundan:montek/issue-4213-configuration-fetch
Open

configuration: replace $.ajax with lorisFetch#10336
Montekkundan wants to merge 3 commits intoaces:mainfrom
Montekkundan:montek/issue-4213-configuration-fetch

Conversation

@Montekkundan
Copy link

Summary

Replace $.ajax calls with lorisFetch in configuration.

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

Follow-up pushed to align with #9999 Client guidance.

Updated

  • Added module client: modules/configuration/jsx/ConfigurationClient.js
  • Migrated process.php requests in modules/configuration/jsx/configuration_helper.js to module client usage.

Notes

  • Existing UI behavior (success/error labels and modal flow) is preserved.

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