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

Add features and prepare for the addition of Barrage, Chapter, and Equipment data functions #1

Draft
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

0t4u
Copy link

@0t4u 0t4u commented Jan 25, 2021

This PR adds filter and search functions for the ship and equipments classes and prepares for the addition of Barrage, Chapter, and Equipment data functions. These changes are tested. Actually adding Barrage, Chapter, and Equipment functions are in the works. Jest tests are also added for automating tests. Please look into commit messages for more details.

…ions

Changed:
- Added barrage, voiceline, chapters to the updater system
- Added barrage. voiceline, chapters sources to SaratogaUtil
- Added barrage, voiceline, chapters to the main Saratoga class
- Changed: how update checking works
Changed:
- Regenerated docma docs
- Minor info addition
Changes:
- Added .filter() and .search() functions for ship data and equipment data
- Regenerate docma docs
@0t4u 0t4u marked this pull request as draft January 25, 2021 10:00
@0t4u 0t4u marked this pull request as ready for review January 25, 2021 10:00
Changed:
- version number (i forgot lmao)
Changed:
- add checks for file exists before accessing file
Changed:
- how checking for updates works
changed:
- add jest test
- add github actions file
@Deivu
Copy link
Member

Deivu commented Jan 26, 2021

On the side note, I will merge this once the barrage, chapter and equip data functions are done

@Deivu Deivu added documentation Improvements or additions to documentation enhancement New feature or request labels Jan 26, 2021
Changes:
 - Comply to: https://docs.google.com/document/d/16rmt309dP2iOlG0tJL2XHgdMGrieUbgmWv-aFVJKmhU/edit
 - Fix jest causing errors in eslint
 - Restructure jest tests
@0t4u
Copy link
Author

0t4u commented Jan 28, 2021

Barrage, chapter and equip data functions are soon TM
Idea: Legacy mode. Basically old Saratoga API type of function naming conventions. If you would like this please write a comment here thanks.

@0t4u
Copy link
Author

0t4u commented Jan 28, 2021

Ill also fix the non-jest vanilla JS tests soon TM

@0t4u
Copy link
Author

0t4u commented Jan 28, 2021

On the side note, there is no eta for barrage, chapter and equip data functions because there are no specs for it yet. If you would like to suggest specs please make changes this doc. Thanks!

Changes:
 - Regenerated docma docs
 - Change version number
Changed:
 - Legacy api support
 - Version number
 - Regenerated docs
@0t4u 0t4u marked this pull request as draft March 9, 2021 05:19
@0t4u 0t4u self-assigned this Mar 9, 2021
0t4u added 4 commits May 10, 2021 18:51
This version adds:
 - Barrage functions
 - Typings for voicelines, barrages, and ships
 - Regenerated docs
 - Partially added JP info
@Deivu
Copy link
Member

Deivu commented May 16, 2021

Legacy class can be removed later

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants