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

Feature/plmc 243 remove tstorageitemid and replace it with hardcoded u32 #75

Conversation

JuaniRios
Copy link
Contributor

No description provided.

@linear
Copy link

linear bot commented Aug 21, 2023

PLMC-243 Remove T::StorageItemId and replace it with hardcoded u32

We already assume we use u32 in the Cleaner, and we agreed internally that 2 Billion items (u32::MAX) is enough for all the projects and project-user items

@JuaniRios JuaniRios self-assigned this Aug 22, 2023
@JuaniRios JuaniRios force-pushed the feature/plmc-243-remove-tstorageitemid-and-replace-it-with-hardcoded-u32 branch 3 times, most recently from 80201b5 to 3f1faba Compare August 23, 2023 14:09
@lrazovic lrazovic changed the base branch from main to feature/plmc-249-remove-option-from-multipliers August 24, 2023 13:42
@lrazovic lrazovic self-requested a review August 24, 2023 13:42
@lrazovic lrazovic changed the base branch from feature/plmc-249-remove-option-from-multipliers to main August 24, 2023 14:24
@JuaniRios JuaniRios force-pushed the feature/plmc-243-remove-tstorageitemid-and-replace-it-with-hardcoded-u32 branch 2 times, most recently from 5e8ed75 to 2f79bde Compare August 25, 2023 09:51
@JuaniRios JuaniRios force-pushed the feature/plmc-243-remove-tstorageitemid-and-replace-it-with-hardcoded-u32 branch from 2f79bde to c8eb63a Compare August 25, 2023 10:20
@JuaniRios JuaniRios merged commit a71eff7 into main Aug 25, 2023
1 of 2 checks passed
@JuaniRios JuaniRios deleted the feature/plmc-243-remove-tstorageitemid-and-replace-it-with-hardcoded-u32 branch August 25, 2023 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants