Skip to content
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

Finally (сколько можно): работающие temp-стенды [2] #147

Merged
merged 11 commits into from
Nov 27, 2020

Conversation

niyazm524
Copy link
Collaborator

may be will close #57

@github-actions
Copy link

github-actions bot commented Nov 27, 2020

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

(expires Thu, 17 Dec 2020 19:37:30 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

@niyazm524
Copy link
Collaborator Author

схожу в магазин и подправлю всё что нужно (пока не ревьювить)

azinit
azinit previously approved these changes Nov 27, 2020
Copy link
Member

@azinit azinit left a comment

Choose a reason for hiding this comment

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

Все огонь, по сложности и замечаниям - в принципе пихуй

Единственное что - вынести бы это все из src и в какую-нибудь root папку положить аля .deploy или .temp-stand

package.json Show resolved Hide resolved
public/dev/main.306cbe88.js Show resolved Hide resolved
src/index.tsx Outdated Show resolved Hide resolved
src/shared/helpers/index.ts Outdated Show resolved Hide resolved
src/shared/helpers/xd-local-storage/index.ts Outdated Show resolved Hide resolved
src/shared/helpers/xd-local-storage/index.ts Outdated Show resolved Hide resolved
src/shared/helpers/xd-local-storage/index.ts Outdated Show resolved Hide resolved
src/shared/helpers/xd-local-storage/index.ts Outdated Show resolved Hide resolved
@azinit azinit added this to the Спринт 2020_6 milestone Nov 27, 2020
@azinit
Copy link
Member

azinit commented Nov 27, 2020

схожу в магазин и подправлю всё что нужно (пока не ревьювить)

Только ща увидел, сорь)

@niyazm524
Copy link
Collaborator Author

схожу в магазин и подправлю всё что нужно (пока не ревьювить)

Только ща увидел, сорь)

а вот кто убрал мой label work in progress?))

@niyazm524
Copy link
Collaborator Author

Из src выгнать не вышло, вебпак не разрешил

@niyazm524 niyazm524 requested a review from azinit November 27, 2020 19:35
@azinit
Copy link
Member

azinit commented Nov 27, 2020

а вот кто убрал мой label work in progress?))
Ахах, так обычно для такого приставляют WIP: префикс

Ну или у нас - указывают проект и в статус в нем

@azinit
Copy link
Member

azinit commented Nov 27, 2020

Из src выгнать не вышло, вебпак не разрешил

Лады, тогда issue (#148) выпишу, и в backlog до лучших времен

Copy link
Member

@azinit azinit left a comment

Choose a reason for hiding this comment

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

Вливай)

@niyazm524
Copy link
Collaborator Author

Из src выгнать не вышло, вебпак не разрешил

Лады, тогда issue (#148) выпишу, и в backlog до лучших времен

там ведь надо eject делать, а так не хочется, так что мб вообще не будем, who knows

@niyazm524 niyazm524 merged commit 2d4568e into dev Nov 27, 2020
@niyazm524 niyazm524 deleted the feature/temp_stands branch November 27, 2020 19:50
@azinit
Copy link
Member

azinit commented Nov 27, 2020

Из src выгнать не вышло, вебпак не разрешил

Лады, тогда issue (#148) выпишу, и в backlog до лучших времен

там ведь надо eject делать, а так не хочется, так что мб вообще не будем, who knows

Ну да, и я об этом же)

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.

Пофиксить авторизацию для temp-стендов
2 participants