Thank you for your contribution.
While I seriously appreciate the work and explanation you have done here, but there were few issues that prevented me from accepting, as follows:
- The key issue would be you missing on clearly describing, and declaring in your code, all the requirements for a properly setup modal without any "warnings". I am a bit surprised as to you showing those screenshots with the missing prop and not even commenting on the topic, or at least defining this prop and avoiding the error.
- I am a bit concerned also with the repository you are always linking to in your different tutorials. It appears you always link to the same repo, yet the repo's age and note states this is for part 3. Could you please clarify for future contributions? The best practice is just to link a complete sample of your related generated code only.
- A less important item would be: Why would you define a password field that is explicit?
- Would be great if you also create more purposeful samples in your tutorials instead of simple basic actions.
- Please also fix the word Curriculum as it's mistyped in all your tutorials.
Need help? Write a ticket on https://support.utopian.io.
Chat with us on Discord.
Hey @mcfarhat, I just gave you a tip for your hard work on moderation. Upvote this comment to support the utopian moderators and increase your future rewards!
Thank you for the suggestion:
I thought of describing the warnings but eventually forgot to keep it. Sorry for that, I have corrected it.
For GitHub concern, I usually commit and update the existing Reacttutorials repository. Also, user can check commit history to get each commit to the respective part of this series and I guess all codes are written well in each part of the series in tutorial itself.
I have defined the password field explicit because in this tutorial my motive was simply to sent multiple user information rather
than single information 'username'. In the next tutorial, I will work on it.
Actually to your fourth suggestion I would like to say that I have added simple basic action because it is difficult for some users to directly understand the complex action. As it may take many days just to understand how data flows and how to add modal, I thought that if I explain to user simple matter first then they could understand easily.
Thank You