Skip to content

Comments

Update: Remove jqueryForm dependency from assetManagementEditAssetView (fixes #343)#336

Merged
taylortom merged 1 commit intoadapt-security:masterfrom
tomwjerry:master
Nov 20, 2025
Merged

Update: Remove jqueryForm dependency from assetManagementEditAssetView (fixes #343)#336
taylortom merged 1 commit intoadapt-security:masterfrom
tomwjerry:master

Conversation

@tomwjerry
Copy link
Contributor

@tomwjerry tomwjerry commented Sep 3, 2025

#343

Update

  • Removed jQuery Form dependency from assetManagementEditAssetView
  • Now uses fetch in its place
  • Changed error callback to reflect the changes
  • Used document.querySelector on one place

@taylortom taylortom changed the title Update: Removed jqueryForm dependency from assetManagementEditAssetView Update: Removed jqueryForm dependency from assetManagementEditAssetView (fixes #343) Sep 10, 2025
@taylortom taylortom moved this from New to Needs Reviewing in adapt-authoring: The TODO Board Sep 10, 2025
@taylortom taylortom changed the title Update: Removed jqueryForm dependency from assetManagementEditAssetView (fixes #343) Update: Remove jqueryForm dependency from assetManagementEditAssetView (fixes #343) Nov 20, 2025
@taylortom taylortom merged commit dc8a779 into adapt-security:master Nov 20, 2025
1 of 2 checks passed
@github-project-automation github-project-automation bot moved this from Needs Reviewing to Merged in adapt-authoring: The TODO Board Nov 20, 2025
github-actions bot pushed a commit that referenced this pull request Nov 20, 2025
# [1.4.0](v1.3.0...v1.4.0) (2025-11-20)

### Update

* Removed jqueryForm dependency from assetManagementEditAssetView (fixes #343) (#336) ([dc8a779](dc8a779)), closes [#343](#343) [#336](#336)
@github-actions
Copy link

🎉 This PR is included in version 1.4.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@taylortom taylortom mentioned this pull request Dec 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Development

Successfully merging this pull request may close these issues.

2 participants