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

Pallets: review if there is a better way to validate user input #203

Open
cernicc opened this issue Aug 14, 2024 · 0 comments
Open

Pallets: review if there is a better way to validate user input #203

cernicc opened this issue Aug 14, 2024 · 0 comments
Assignees
Labels
enhancement New feature or request pallet-market Relates to the Market Pallet pallet-storage-provider Relates to the Storage Provider Pallet tech debt Flags technical debt
Milestone

Comments

@cernicc
Copy link
Member

cernicc commented Aug 14, 2024

This issue was created due to the debate here. We debated where should we check if the sectors passed by the users are empty.

The overall issue is how and where to validate user input on the extriniscs. Currently, we are checking things like, does the sector exists for some storage provider on multiple different locations. Where and how should this be handled?

@cernicc cernicc added enhancement New feature or request pallet-storage-provider Relates to the Storage Provider Pallet pallet-market Relates to the Market Pallet labels Aug 14, 2024
@jmg-duarte jmg-duarte added this to the Phase 3 milestone Nov 14, 2024
@jmg-duarte jmg-duarte added the tech debt Flags technical debt label Nov 15, 2024
@cernicc cernicc self-assigned this Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pallet-market Relates to the Market Pallet pallet-storage-provider Relates to the Storage Provider Pallet tech debt Flags technical debt
Projects
None yet
Development

No branches or pull requests

2 participants