-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat/organizations page #25
Conversation
…ds; start writing test-data object
… handler to component
@ssagg, смотри, |
принял |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
так же не прошли чеки
app/fragments/organization-card/src/organization-card.component.tsx
Outdated
Show resolved
Hide resolved
app/fragments/organization-card/src/organization-card.component.tsx
Outdated
Show resolved
Hide resolved
d4fb92b
to
87e9762
Compare
@ssagg обновил тему |
я вот здесь лейаут для страниц добавл. у меня почему-то если роуты |
@effektivnayarabota1 да, если app роутинг использовать, то надо page все называть. Переименовал свои страницы тоже. Предлагаю папку management переименовать в dashboard, это же в пути в браузере будет. В крайнем случае скобками потом изолируем. Посмотри мой послдений коммит можешь скачать. там по дефолту идет редирект на dashboard/users и по сслыке registration вторая страница. потом напишем мидлвару авторизациии и protected route. еще надо решить по стилям потом. в лэйауте их оставить или во фрагментах. |
app/entrypoints/renderer/src/app/management/organizations/page.ts
Outdated
Show resolved
Hide resolved
dashboard - ok на счет стилей не понял. ты про |
например, есть марджин в бейс лайоут и есть марджин в юсер-тайтл и в тем-селектор. но возможно у тебя во фрагментах исправлено. base-layout незначительно поправил, т.к. элемент неправильно отображались |
у меня наржинов нет. есть паддинг общий на всю обертку + gap. на счет круглых скобок - давай так сделаем, логично все. |
Closes #23