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

feat: add building utilities for extra #93

Merged
merged 3 commits into from
Mar 8, 2023

Conversation

spaenleh
Copy link
Member

@spaenleh spaenleh commented Mar 3, 2023

This PR adds building methods to create item extras.

Will be useful to remove some duplicated utils from builder and player.

closes #92

@spaenleh spaenleh added the chore Generic task or chore label Mar 3, 2023
@spaenleh spaenleh self-assigned this Mar 3, 2023
@spaenleh spaenleh linked an issue Mar 3, 2023 that may be closed by this pull request
@spaenleh spaenleh marked this pull request as draft March 3, 2023 10:15
src/utils/extra.ts Outdated Show resolved Hide resolved
@spaenleh spaenleh force-pushed the 92-add-build-utilities-for-extra branch from 0548ca8 to 0c1a7ee Compare March 7, 2023 14:29
@spaenleh spaenleh marked this pull request as ready for review March 8, 2023 11:11
@spaenleh spaenleh requested review from pyphilia and codeofmochi March 8, 2023 11:11
src/utils/extra.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@codeofmochi codeofmochi left a comment

Choose a reason for hiding this comment

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

Let me know if the suggested changes make sense to you. Otherwise I'll be happy to approve 👍

spaenleh and others added 2 commits March 8, 2023 13:54
@spaenleh
Copy link
Member Author

spaenleh commented Mar 8, 2023

@dialexo Thank you for the suggestions, I added both as they make sense.

@spaenleh spaenleh requested a review from codeofmochi March 8, 2023 12:56
@spaenleh spaenleh merged commit 7eba62a into main Mar 8, 2023
@spaenleh spaenleh deleted the 92-add-build-utilities-for-extra branch March 8, 2023 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Generic task or chore
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add build utilities for extra
2 participants