Follow-up from "Resolve "Implement upgraded version of forms "informed"""
The following discussions from !2 (merged) should be addressed:
-
@sam started a discussion: It would be good to have a comment here to discuss purpose. Also can we put in a
propType
for this prop too? -
@sam started a discussion: Comment needed for this method.
-
@sam started a discussion: Comment needed for this method.
-
@sam started a discussion: A comment above this line explaining why this value should be deleted might be useful for future maintainers.
-
@sam started a discussion: This file needs comments throughout. Comments for
propTypes
, and methods. -
@sam started a discussion: Needs method comments, and would be good to get propTypes specific to this component added, if there are any.
-
@sam started a discussion: Needs method comments.
-
@sam started a discussion: Needs method comments, and remove extra
console.log
-
@sam started a discussion: Comments throughout (I see defaults above are commented which is great, but need propTypes & methods commented too).