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

fix(world): Items and traits duplication issue on encyclopedia #913

Merged
merged 2 commits into from
Apr 18, 2024

Conversation

SpeedyD
Copy link
Contributor

@SpeedyD SpeedyD commented Apr 17, 2024

On LKs with high amounts of items/traits, eventually items/traits start duplicating over pages. An item/trait that was halfway page 8 is also at the top of page 9.

And worst of all: When you get to the last page, you definitely have not seen all items/traits. Some of them have just poofed into the ether.

With this ingenious solution I have finally prevented this duplication!
(Note: solution is actually really, really dumb. I can't believe this fixed it.)

@itinerare itinerare added bug Something isn't working needs review Pull requests that are pending community review labels Apr 17, 2024
@itinerare
Copy link
Collaborator

I'd assume it can occur for most objects in /world, then, it just usually isn't a trouble/they usually aren't in significant enough numbers for the issue to manifest.

@SpeedyD
Copy link
Contributor Author

SpeedyD commented Apr 17, 2024

I'd assume it can occur for most objects in /world, then, it just usually isn't a trouble/they usually aren't in significant enough numbers for the issue to manifest.

This can ABSOLUTELY occur for most objects in /world - I'm in fact surprised it's not doing it on the masterlist or userlists.

I just figured that Currencies, Rarities, Species, Species Categories, Trait Categories, Subtypes and Item Categories are unlikely to hit the same amount.

Even if so.. the same fix can be easily applied.

@SpeedyD
Copy link
Contributor Author

SpeedyD commented Apr 17, 2024

...and so I did. Seems like a good idea.

@itinerare itinerare added reviewed Pull requests that have received community review and are pending merge and removed needs review Pull requests that are pending community review labels Apr 18, 2024
@itinerare itinerare merged commit 8dc56e4 into corowne:release/v3.0.0 Apr 18, 2024
0 of 2 checks passed
@SpeedyD SpeedyD deleted the fix/items-trait-page-dupe branch April 18, 2024 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working reviewed Pull requests that have received community review and are pending merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants