Skip to content
This repository has been archived by the owner on Sep 5, 2019. It is now read-only.

Pilot Fit Scan tool + basic groundwork for pilot settings #106

Merged
merged 5 commits into from
May 2, 2018
Merged

Conversation

samuelgrant
Copy link
Collaborator

  • Added a ship fit scan tool for FCs located under the FC Tools menu.

  • Moved the non-existent link for pilot search and added a non-existant link for invite logs to the FC Tools menu.

  • Added controllers and commented routes for pilot settings, invite logs fc tools.

  • Added a static view for pilot settings. This can be used by @Makeshift when he starts the alt management module.

  • Extra CSS

  • Fixed the listener issue [Bug] EventEmitter Memory leak #102

  • Removed some unneeded JS [Query] Possible redundant code. #104

- Removed an unneeded jquery script from the head of each page
- Added new routes for fcTools and pilotSettings (fitting is active the others are uncommented)
- Added verious controllers for fcTools and pilotSettings
- Made some basic CSS adjustments
- Adjusted spacing in some of the views
- Added toolsFits.njk for ship scanning

Tool has been taken from: http://fitscan.nakamura-labs.com/ . Multiple ships are out of date and it was easier for us to update them. This page will be removed when CCP gives us the active fit endpoint.
- Removed the redundant code in (Issue #104)[#104]
- Set maxListners(0) (Issue #102)[#102] turns out it actually works/
Fixed an issue that stopped the FC Tools and Waitlist admin menus from opening correctly.
- Updated ships.json so that it only has the ships we use and the slot loadouts are correct.
- Updated the view so that a dropdown box is used to select the ship rather than a type box. This allows us to get rid of typeahead.
- Added a copy to clipboard button for fits.
- Active wing filter is no longer case sensitive
- Notification bug #103 resolved
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants