Skip to content

Conversation

silasary
Copy link
Collaborator

This is useful both for providing buffer space for future expansion (without shifting later IDs), and also just nice for organization reasons.

It's also an alternative to #73

@axxroytovu
Copy link
Contributor

Can you add a syntax example to the Items.json file?

@nicopop
Copy link
Contributor

nicopop commented Sep 19, 2024

  1. this probably need to be added to the json schemas
  2. might cause id issue down the line if an id is specified and later the automated count get counted up to it

@silasary
Copy link
Collaborator Author

  1. this probably need to be added to the json schemas

Yup, can do.

  1. might cause id issue down the line if an id is specified and later the automated count get counted up to it

This is why I throw an error if that happens.

nicopop
nicopop previously approved these changes Sep 23, 2024
Copy link
Contributor

@nicopop nicopop left a comment

Choose a reason for hiding this comment

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

I dont think its as good as it could be since this wont allow the ID to go back at all but it should be good enough for now

@FuzzyGamesOn FuzzyGamesOn merged commit c38893a into main Sep 24, 2024
@silasary silasary deleted the skip-ids branch December 2, 2024 04:13
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.

4 participants