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

More failsafe demo maker #32

Open
sveetch opened this issue Jan 1, 2022 · 0 comments
Open

More failsafe demo maker #32

sveetch opened this issue Jan 1, 2022 · 0 comments
Labels
Enhancement Feature enhancement

Comments

@sveetch
Copy link
Member

sveetch commented Jan 1, 2022

The lotus_demo command is making a lot of computation with articles and so require a minimum article length, however this is not enforced by validation or documentation.

Let's add some argument validation and improve command doc helps.

  • Minimal article length is around 7;
  • Add some mention about the factory usage which is not totally safe and can lead sometime to failure on unique constraints (like sometime the Author factory can try to use the same user name twice);
@sveetch sveetch added Documentation Improvements or additions to documentation Enhancement Feature enhancement labels Jan 1, 2022
@sveetch sveetch removed the Documentation Improvements or additions to documentation label Dec 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Feature enhancement
Projects
None yet
Development

No branches or pull requests

1 participant