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

Fix: update packages #404

Merged
merged 2 commits into from
Mar 15, 2021
Merged

Fix: update packages #404

merged 2 commits into from
Mar 15, 2021

Conversation

pavolzbell
Copy link
Contributor

No description provided.

@pavolzbell
Copy link
Contributor Author

@jsuchal pustil som yarn upgrade ale nie som si isty ci nieco taketo mozeme len tak mergnut? mozno by to chcelo nejaky vizualny test

@pavolzbell pavolzbell requested a review from jsuchal March 9, 2021 10:19
@jsuchal
Copy link
Member

jsuchal commented Mar 9, 2021

Tu by som bol fakt opatrnejsi, ja by som preklikal nejake JS veci (prihlasovanie do newslettra, rozbalovanie faq, klikanie na steps ako hotove). Aj update frontendu je zrely na sync s upstream (gov.uk repo) ale tam je vela konfliktov zatial to nikto nedal aj ked pokusy boli. slovensko-digital/navody-frontend#124

@jsuchal jsuchal self-assigned this Mar 12, 2021
@michal-rohacek
Copy link
Contributor

presiel som vsetko vyssie uvedene + feedbacky a captchy a vyzerame ze je vsetko okej.

@jsuchal
Copy link
Member

jsuchal commented Mar 15, 2021

@michal-rohacek tak mergujme a skusame

@michal-rohacek michal-rohacek merged commit 239b679 into master Mar 15, 2021
@michal-rohacek michal-rohacek deleted the fix/packages branch March 15, 2021 21:17
@jsuchal
Copy link
Member

jsuchal commented Mar 15, 2021

@michal-rohacek pozeram dobre ze tu nezbehol deploy na staging po merge?

@michal-rohacek
Copy link
Contributor

@jsuchal vyzera ze nic nezbehlo.pravda

@michal-rohacek
Copy link
Contributor

hodim to manualne a do jiry

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.

3 participants