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

Adds import/export decks feature #237

Merged
merged 1 commit into from
Nov 10, 2022

Conversation

willroberts
Copy link
Collaborator

@willroberts willroberts commented Nov 10, 2022

Summary

Adds "Export Decks" and "Import Decks" buttons to the Collection screen. "Export Decks" copies decks JSON to the user's clipboard, and "Import Decks" provides a text box to paste JSON into.

Closes #224.

App changes (app/, etc.):

  • Adds exportDecks and importDecks functions to InventoryManager
  • Adds import/export buttons and handlers to the Collection view
  • Adds new dialog windows for importing and exporting decks
  • Tweaks startup logging a bit
  • Comments more unreachable code

Server changes (config/, server/, worker/, etc.):

  • Adds getDecksAsJSON and putDecksFromJSON methods to DecksModule
  • Adds /api/me/decks/export-decks and /api/me/decks/import-decks routes
  • Removes the custom StackLogger class

Build changes (docker/, gulp/, scripts/, etc.):

  • Removes the pretty-error dependency
  • Speeds up localdev rebuilds by retaining node_modules

Testing

Have you have tested your changes in the following scenarios?
Feel free to check off scenarios which don't apply.

  • Starting backend services locally with docker compose up succeeds.
  • I am able to create a new user and log in locally.
  • I am able to complete a practice game locally.
  • I am able to complete a purchase of Orbs, etc.

@willroberts willroberts merged commit f2b53c0 into open-duelyst:main Nov 10, 2022
@willroberts willroberts deleted the export-decks branch November 10, 2022 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[P0] Allow export and import of decks
1 participant