-
Notifications
You must be signed in to change notification settings - Fork 215
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
Store sale offer events for marketplace sale-item collection. #27
Conversation
Upon further consideration, we should not be querying over all events, because performance will degrade, the more users are sending events to the system. Going to try to only keep records of sales in the DB by adding/removing based on events received. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good. Going to run through the setup one more time before approving 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Something strange is happening when I test out the FE app.
After listing an item for sale:
- The "List" button changed to "Unlist" but the spinner never stops
- My laptop fans starting ramping up. I tried to switch to the "Marketplace Items" tab -- it was very slow to change tabs, and after a few seconds Chrome disabled the tab
- I then tried reloading the page in Firefox too, but it also froze and the tab was killed by FF
Any chance there's a runaway loop in the frontend code?
148341c
to
c5aa0ff
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is no way to see all items available for sale in the marketplace.
This PR is an attempt to create such a list by capturing sale-offer events using the events worker.
This PR also removes the
kitty_items
table.