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

inventory frontend rewrite #17

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

Conversation

shocktail39
Copy link

currently, this is what the code for the inventory's frontend looks like:

screenshot of obfuscated code

this is basically unmaintainable. i'm trying to rewrite the inventory's frontend to be readable, and maybe i'll actually finish a project this time. this is not even close to feature complete yet, so hold off on merging.

@shocktail39 shocktail39 changed the title started inventory frontend rewrite. display saved items and folders. inventory frontend rewrite Jan 6, 2023
@shocktail39
Copy link
Author

putting some notes here for reference. there are four commands that the backend sends to the frontend:

  • script-to-web-inventory
  • script-to-web-settings
  • script-to-web-receiving-item-queue
  • script-to-web-nearby-users

and there are eight commands that the backend listens for:

  • ready
  • web-to-script-inventory
  • web-to-script-settings
  • use-item
  • share-item
  • web-to-script-nearby-users
  • web-to-script-request-receiving-item-queue
  • web-to-script-update-receiving-item-queue

currently, my rewrite sends ready and handles script-to-web-inventory. any other signals sent to it currently just get put in an alert.

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.

2 participants