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.

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

Remove jQuery #159

Open
3 tasks
misscs opened this issue Oct 1, 2017 · 2 comments
Open
3 tasks

Remove jQuery #159

misscs opened this issue Oct 1, 2017 · 2 comments

Comments

@misscs
Copy link
Contributor

misscs commented Oct 1, 2017

Vision

Reduce dependencies and leverage native browser APIs.

Current State

Code base is using jQuery just to make Ajax calls (net.js). These could easily be replaced with Fetch, Async/Wait, or a single purpose node module, such as request module, that's already in the code base.

Scope

  • Clarify app support requirements and goals to determine best approach
  • Rewrite ajax calls in net.js to use agreed upon approach
  • Remove jQuery (profit & 💃 )
@misscs misscs changed the title [clientAdmin] Remove jQuery Remove jQuery Oct 1, 2017
@patcon
Copy link
Contributor

patcon commented Oct 19, 2017

As someone who's never exorcised jQuery before, i was curious and dug a little more (in case someone else wants to run with this):

jQuery Methods used (grepped $\.):

  • $.get()
  • $.Deferred()
  • $.ajax()
  • $.when()
  • $.extend()

Re: app support environment. Openned #166

@patcon
Copy link
Contributor

patcon commented Jul 30, 2020

Related issue: #364 (possibly dup)

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

No branches or pull requests

2 participants