Skip to content

feat(sidebar): refactored module (#DS-3649) #736

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

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Conversation

artembelik
Copy link
Contributor

@artembelik artembelik commented May 16, 2025

  • standalone
  • signals
  • docs
  • api + description
  • examples
  • units

#758

@artembelik artembelik self-assigned this May 16, 2025
ru: 'Sidebar',
en: 'Sidebar'
},
svgPreview: 'sidebar',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@rmnturov нужна svg иконка

Copy link

github-actions bot commented May 16, 2025

Visit the preview URL for this PR (updated for commit c9b0ae1):

https://koobiq-next--prs-736-omipqb7h.web.app

(expires Sun, 25 May 2025 09:32:33 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c

@lskramarov
Copy link
Contributor

Ломающих изменений не должно быть

@artembelik artembelik marked this pull request as ready for review May 19, 2025 14:37
@artembelik artembelik requested a review from lskramarov as a code owner May 19, 2025 14:37
@artembelik artembelik requested a review from NikGurev May 19, 2025 14:37
@artembelik
Copy link
Contributor Author

Ломающих изменений не должно быть

публичное апи осталось прежним, изменилась только внутренняя реализация

@lskramarov
Copy link
Contributor

Ломающих изменений не должно быть

публичное апи осталось прежним, изменилась только внутренняя реализация

а это
image

задача не предполагала рефакторинг:
image

@artembelik artembelik marked this pull request as draft May 20, 2025 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants