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 CSV exports & massively improve scaling #163

Merged
merged 2 commits into from
Aug 3, 2023

Conversation

JSKitty
Copy link
Member

@JSKitty JSKitty commented Jul 31, 2023

Abstract

This is the first of a handful of likely PRs to improve PIVX Promos, using feedback from the release of v1.0

This PR focuses on the scalability of MPW's PIVX Promos implementation, massively improving it's usage in larger scopes, such as mass-generated codes or organised and automated CSV sheet exports.

image

image

Promo codes are now stored in Memory rather than only on Disk, persisting their state between 'update ticks', which allows us to render Promos prior to their state being synced (shown as 'Syncing') and also allows multithreaded synchronisation of the codes, handled by the browser.

What does this PR address?

PIVX Promos in MPW could previously slow down a lot under heavy generation, this is resolved by this PR, this also resolves the issue of users having to manually export each code individually: they may now use a CSV sheet to export the entire PIVX Promos list in one click, with an easy-to-read, informative format on each code and their live balances.

What features or improvements were added?

A CSV export for PIVX Promos, alongside massive scaling improvements that lessen loading times and enhances the app's responsiveness.

How does this benefit users?

Faster loading, less waiting, and less manual work by having to export codes by-hand, PIVX Promos can now be partially automated.

@JSKitty JSKitty added the Enhancement New feature or request label Jul 31, 2023
@JSKitty JSKitty requested review from Liquid369, Duddino, PalmTreeVI, BreadJS and a team July 31, 2023 10:27
@JSKitty JSKitty self-assigned this Jul 31, 2023
PalmTreeVI
PalmTreeVI previously approved these changes Jul 31, 2023
Copy link

@PalmTreeVI PalmTreeVI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tACK - worked as expected, was easily imported into a spreadsheet.

Copy link

@Liquid369 Liquid369 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tACK d20e65c

Testnet operated fine, normal functions were still intact, pretty little CSV.
Screenshot 2023-08-01 at 1 05 41 PM

@JSKitty JSKitty requested a review from PalmTreeVI August 1, 2023 23:42
@JSKitty JSKitty merged commit f4b5eaa into master Aug 3, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
Development

Successfully merging this pull request may close these issues.

3 participants