ToDo: Refactor JS FIle - Githubissues

jwilferd10 / Zap-and-Drive

As electric cars continue to gain traction, the need for convenient EV Charging Stations has never been greater. Allow Zap & Drive to find you those stations and ⚡ ZAP ⚡ power into your battery!

https://jwilferd10.github.io/Zap-and-Drive/

0 stars 0 forks source link

ToDo: Refactor JS FIle #4

Closed jwilferd10 closed 6 months ago

jwilferd10 commented 1 year ago

Remove redundant comments
Rename functions to remain organized
Refactor core concepts to perform optimally

jwilferd10 commented 1 year ago

The main functionality will revolve around using API's.

Rename the script file into app/main
Modularize code by creating new js files
- Keep each file specific to a functionality
Export/Import content into the main js

jwilferd10 commented 1 year ago

Commit dd0b632 Update

JS File has been completely refactored and pushed onto the main branch.

This has knowingly created some bugs, but the code is more readable and easier to break apart and work on.

Future updates will target the functionality and ensure it works.

jwilferd10 commented 1 year ago

Commit commit 551529c Update

Reverted and refactored the JS file, the project is now functional
- Main changes center around the functionality, it's been rewritten to be more easily read
- New functions have been created hold the logic of previously larger functions. Should be easy to edit (Ex: getEVMap)

Future changes are going to revolve around adding comments and further fine-tuning.

jwilferd10 commented 6 months ago

Commit 38d87f0 Updates

Removed Additional Info section within index.html
Removed both getGeoLocation() & getLocation's functionality that populated innerHTML elements with data

jwilferd10 commented 6 months ago

Commit 047147b Updates

Deletion of markers/icons from map now functional
- Changes were made to removeMarkers() method a few commits before this specific commit
New createMarker() method has been added to handle the markers/icon functionality
Refactored handleChargeStationData() to use createMarker() method

jwilferd10 commented 6 months ago

To-Do: Update script.js to utilize more ES6 functionality

Apply async/await for asynchronous operations
Try and Catch for error handling

jwilferd10 commented 6 months ago

Parameters for this issue have been met