Skip to content

Conversation

@nathalieMalsh
Copy link
Contributor

Что сделано:

  • создана страница "Программы обучения" для админ-панели (pages/Admin/Programs/Index.tsx)
  • реализован виджет study-programs
  • реализована интеграция текста компонента с i18n

Ссылка на задачу: #909

import { StudyPrograms } from "@widgets/study-programs"

// Программы для демонстрации
const mockPrograms = [
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В проект уже добавлен MSW, можно перенести моки туда, вот тут можно посмотреть пример https://github.com/aisaenok/hexlet-cv/blob/issues-897/frontend/src/mocks/account/index.ts

}

return (
<Container size='xl' py='md' bg='white'>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Опять же макет примерный, не нужно указывать white black и т д, делаем все на темной теме она по умолчанию уже прописана в тем провайдере. Т е убери везде bg= c= и т п

const { t } = useTranslation()

// стили прописаны временно для разработки
const inputStyles = {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Тоже можно убрать

</Title>
<Group gap='xs' mb='md'>
{ /* Пока не знаю, что должна делать эта кнопка */ }
<Button type='submit' variant='default'>{t('adminPage.studyPrograms.button')}</Button>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

А кнопка почему не реализована? На это другая задача есть?

{programs?.map((program) => (
<Table.Tr key={program.id}>
<Table.Td>
<TextInput
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно ли редактировать? Если да, то опять же в этой же задаче должно быть сделано?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants