небольшие комментарии

[atherdon]

1. можем ли мы вынести этот большой объект outside - мне просто ненравится когда внутри компонента все лежит.
   https://github.com/atherdon/subscribe-form/blob/master/components/forms/subscribe.js#L14

пример из туториала который я нашел

const LoginSchema = Yup.object().shape({
  email: Yup.string()
    .email("Invalid email address format")
    .required("Email is required"),
  password: Yup.string()
    .min(3, "Password must be 3 characters at minimum")
    .required("Password is required")
});

<Formik
              initialValues={{ email: "", password: "" }}
              validationSchema={LoginSchema}

2. сделать отдельным компонентом. можем ли мы заюзать компонент, который мы делали в sendgrid-forms? https://github.com/atherdon/subscribe-form/blob/master/components/forms/subscribe.js#L53

3. использовать email component для обоих форм

4. вынести чекбоксы в отдельные файлы. я хочу потом эти checkboxes re-use
   https://github.com/atherdon/subscribe-form/blob/master/components/forms/subscribe.js#L86
   https://github.com/atherdon/subscribe-form/blob/master/components/forms/subscribe.js#L92
   https://github.com/atherdon/subscribe-form/blob/master/components/forms/subscribe.js#L98
   https://github.com/atherdon/subscribe-form/blob/master/components/forms/subscribe.js#L104
   https://github.com/atherdon/subscribe-form/blob/master/components/forms/subscribe.js#L110

сс @coder-do

[coder-do]

Первые 3 сделал

насчёт 4ого - у нас и так есть отдельный компонент чекбоксов, если имеете ввиду что еще эти 5 вместе вынести куда то - сделаю

[atherdon]

ну у нас хоть и компонент отдельный, все равно параметры передаются тут.

причина по которой я хочу вынести их отдельно - чтобы форма была более flexible. плюс позже, мы сможем это использовать в #56

там же тоже есть чекбоксы, которые относятся к типам рассылок