adding scenarios and adding screen

[BrunnaSilva]

No description provided.

[aryella-lacerda]

Quando a gente sobe algum código pra develop ou pra master, tentamos sempre subir com fixtures desativadas. Isso evita eventuais erros na hora de gerar versões de produção.

[aryella-lacerda]

Para tipagem de funções, você pode definir assim:

type Props = {
    item: ToDoAdd,
    onToDos: (values: ToDoAdd) => void
}

• Tente definir nomes com verbos para callbacks. Por exemplo, em vez de onToDo, definir onAddToDo ou onAddItem. Como você escolheu o termo item no prop acima, era legal padronizar.

[aryella-lacerda]

De onde veio esse ref? 🤔

[aryella-lacerda]

Por que usar o TextInput do react-native-gesture-handler e não o do react-native? 🤔 A não ser que ele faça alguma coisa especial, é mais seguro usar os componentes 'nativos' ao react-native.

[aryella-lacerda]

Numa PR normal, comentários ou console.logs no código geralmente fazem a tarefa voltar, pra você retirar.

[aryella-lacerda]

Com propriedades booleanas, é recomendado seguir o padrão de adicionar um is ou are no início da frase. Por exemplo:

const [isAdding, setIsAdding] = useState(false)
const [isOpen, setIsOpen] = useState(false) 

ou se você preferir (com tipagem do Typescipt):

const [isAdding, setAddingStatus] = useState<boolean>(false)
const [isOpen, setOpenStatus] = useState<boolean>(false) 

[gabrielkabral]

E seria interessante também deixar mais claro o que está sendo adicionado, aberto ou o que a data representa, por exemplo:

const [isAddingSomething, setAddingSomething] = useState<boolean>(false)
const [isSomethingOpen, setIsSomethingOpen] = useState<boolean>(false)
const [reminderDate, setReminderDate] = useState<Date>(new Date())

[aryella-lacerda]

Esses componentes dentro desse arquivo são bem pequenos, mas ainda assim, merecem seu próprio arquivo, seu próprio arquivo de estilos, seu próprio registro no Storybook, etc. Inclusive, são esses componentes pequenos que são super reutilizáveis e não podem estar perdidos dentro de um arquivo de tela.

[aryella-lacerda]

Um erro bastante comum é guardar informações de estado de dados dentro do componente e não da tela. Por exemplo, a tela de "Novo Lembrete" precisa da prioridade, mas esse informação á perdida aqui dentro desse componente. Para isso, a gente geralmente usa o padrão controlled-component, onde a tela controla o componente. Ficaria algo assim:

const ToDoScreen = () => {
  const [selectedPriority, onSelectPriority] = useState('Selecionar')
  return (.... 
     <PriorityList 
        selectedPriority={selectedPriority} 
        onSelectPriority={onSelectPriority} 
     /> 
  ...)
}

const PriorityList = ({ selectedPriority, onSelectPriority }) => (...)

[aryella-lacerda]

Um ToDo é um item, não uma ação, então não precisa do "Add" no final.

1. Existe um ToDo que não tem título? Até consigo imaginar não ter lembrete ou prioridade, mas título é importante.
2. Geralmente não colocamos o | null direto na entidade, pq aí, em todo local que ele é usado no app, ele poderá ser nulo, e isso não é o que queremos. Em lugares específicos, e gente pode adicionar o | null mas não na definição da entidade.

[aryella-lacerda]

Por que precisou adicionar isso? 🤔