-
Notifications
You must be signed in to change notification settings - Fork 138
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
Unclaimed codes #932
Merged
Merged
Unclaimed codes #932
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
DonKoko
commented
Apr 26, 2024
•
edited
edited
- made it so userId and organizationId are not required for qr codes
- updated some services that needed change due to different fields
- created new services for generating unclaimed codes
- created admin dashboard index for qrs
- created endpoint to generate and print unclaimed codes
- created a util that takes an array of Prisma.Qr and generates a zipBlob with svgs of those codes
- changed batch from a field within Qr to its own model with Relation to Qr
- updated qr codes index for admin to have better data
- added filter based on batch name/id
- added a form that allows to mark a batch as printed
- made it so userId and organizationId are not required for qr codes - updated some services that needed change due to different fields - created new services for generating unclaimed codes - updated DB to include 'printed' and 'batch' fields for qr codes - created admin dashboard index for qrs - created endpoint to generate and print unclaimed codes - created a util that takes an array of Prisma.Qr and generates a zipBlob with svgs of those codes
- changed batch from a field within Qr to its own model with Relation to Qr - updated qr codes index for admin to have better data - added filter based on batch name/id - added a form that allows to mark a batch as printed
- updating organization-select to make it re-usable - updating flow after scanning a qr to handle unclaimed codes - updated the flow of qrId.link to handle linking and claiming codes, aside from only linking - updated contact-owner to handle the case when someone lands there for a code which doesnt have an owner
|
…n component to use setSearchParams
@jurrejansen issue is fixed. Deploying now: https://github.com/Shelf-nu/shelf.nu/actions/runs/8893042370 I also improved the pagination component cuz i found another bug and improved the code to make it smaller and smoother. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.