Fix for #1348 Add a timer to the tab with the solution #1499

antonDedyaev · 2023-02-15T12:21:22Z

Added a timer to the Solution tab that hides teacher's solution until the countdown is over. As soon as the countdown is over, the "show solution" button pops out and the user can click it to see the solution.

fey · 2023-02-16T11:33:40Z

public/images/waiting_clock.png - это лишнее. Посмотри, как у нас в файле mix.js вроде бы сделано - там копируются картинки из ассетов в в паблик директорию с помощью вебпака. Даже скорее эти картинки уже копируются.

antonDedyaev · 2023-02-16T11:48:43Z

Она из ассетов и подтянулась видимо, так как я туда ее и поместил рядом с другими картинками. Как исправить? Если ее удалить из ассетов, она ведь и из паблика удалится тогда, не?

fey · 2023-02-16T14:24:09Z

нужно удалить только из public/
все файлы ассетов внутри директории public/ мы игнорируем. Мб какой-то не тот путь, посмотри как лежат другие файлы.
Удалить можно git rm <path>

antonDedyaev · 2023-02-16T19:32:01Z

Пути проверил. Картинка с часами, как и все прочие лежит в папке assets/img. В public/ вся директория img игнорируется, в том числе и эта картинка. Но... там же в public есть еще одна папка - images, там по какой-то причине эта картинка тоже дублируется, а если удалить эту папку, то она снова появляется как только делаешь make start...
Могу заигнорить ее через gitignore. Или это плохое решение?

fey · 2023-02-16T20:40:54Z

Поменяй путь на public/img/картинка

fey · 2023-02-16T20:41:28Z

суть в том, что картинки в паблике игнорятся как и любые ассеты, которые собираются. А у тебя путь не тот указан, в котором все картинки лежат

antonDedyaev · 2023-02-16T20:55:19Z

А где путь то менять? В импорте картинки в компоненте? У меня сейчас
import waitingClock from '../../assets/img/waiting_clock.png'.
Его менять надо? Или я уже туплю под ночь?)
Я просто не пойму откуда возникает эта папка images - я нигде ее не указывал....

fey · 2023-02-16T21:02:58Z

Тогда попозже гляну в чем дело

ashikov · 2023-02-20T14:14:27Z

Было время, глянул. Во-первых, git rm ... работает нормально, потом просто нужно коммитнуть это удаление. Вот в ветке я это сделал: ea6525e
@antonDedyaev смотри здесь у нас ассеты копируются при сборке фронта из resources/assets/img в public/img. На проде оно всё находится в public/img, после сборки. Соответственно путь до ассета должен указывать на public/img. Это я про вот эту часть:

hexlet-sicp/resources/js/components/TeacherSolution.jsx

Line 10 in ea6525e

import waitingClock from '../../assets/img/waiting_clock.png';

antonDedyaev · 2023-02-20T20:02:46Z

Было время, глянул. Во-первых, git rm ... работает нормально, потом просто нужно коммитнуть это удаление. Вот в ветке я это сделал: ea6525e @antonDedyaev смотри здесь у нас ассеты копируются при сборке фронта из resources/assets/img в public/img. На проде оно всё находится в public/img, после сборки. Соответственно путь до ассета должен указывать на public/img. Это я про вот эту часть:

hexlet-sicp/resources/js/components/TeacherSolution.jsx

Line 10 in ea6525e

import waitingClock from '../../assets/img/waiting_clock.png';

Привет, @ashikov !
Спасибо за ответ. Тогда смотри - чисто для уточнения:
Достаточно ли мне будет изменить путь в импорте на '../../../public/img/waiting_clock.png' ?
Если да, то после коммита в репо, мне нужно будет делать еще один ПР - или коммит зайдет в текущий ПР?

И еще все равно не могу понять, откуда берется папка public/images ??? Нигде в конфигах ее не нашел, в webpack.mix как раз речь только про копирование в public/img...
И, кстати, даже после изменения пути, если я запускаю сборку через make start - эта папка public/images снова генерируется с этой же картинкой. Что за мистика?)

fey · 2023-02-20T20:30:17Z

resources/js/components/TeacherSolution.jsx

+import { useTranslation } from 'react-i18next';
+
+import { changeShowStatus } from '../slices/solutionSlice';
+import waitingClock from '../../assets/img/waiting_clock.png';


вот из-за этого импорта картинка попадает из ассетов в public/images
Скорее всего это при компиляции происходит, одним из плагинов, например SassLoader

fey · 2023-02-20T20:31:24Z

собственно выше написал, это какой-то из плагинов срабатывает, который тянет картинку из ЖС и публикует её.
Поступим проще - поменяем директорию с картинками, вместо img будет images
в мейн запушу, тебе нужно будет освежить свою ветку

ashikov · 2023-02-21T03:23:19Z

после коммита в репо, мне нужно будет делать еще один ПР - или коммит зайдет в текущий ПР?

Коммиты, отправленные в ветку форка из которой был создан PR, попадают в этот пулреквест автоматически

antonDedyaev · 2023-02-21T09:07:19Z

Обновил форкнутую ветку main, потом сделал в своей ветке рибейз. Возник конфликт - вручную поправил. Сделал коммит. Поменял путь импорта, как согласовали выше, потом пуш - теперь смотрю, что некоторые коммиты продублированы, но с другим хэшем.
Может отменить этот ПР и сделать новый ?

fey · 2023-02-21T21:31:20Z

Попробуй сделать ребейз на мейн. Там будут конфликты, попробуй их порешать.
Если не получится - можем созвониться. Делать новый ПР всегда можно успеть. а попрактиковаться с конфликтами всегда полезно.

antonDedyaev · 2023-02-22T07:03:10Z

Попробуй сделать ребейз на мейн. Там будут конфликты, попробуй их порешать.
Если не получится - можем созвониться. Делать новый ПР всегда можно успеть. а попрактиковаться с конфликтами всегда полезно.

Рибейз на локальный мейн или на удаленный?

fey · 2023-02-22T16:58:26Z

Поскольку твоей мейн должен быть 1 в 1 с удаленным Хекслета (т.е. ты сперва делаешь на мейне git pull --rebase upstream main, а ребейз).
В итоге комада ребейза будет такая git rebase -i main, если я не ошибаюсь.

antonDedyaev · 2023-02-22T21:09:18Z

@fey Сделал рибейз, и в интерактивном режиме удалил "дубликаты коммитов", затем разобрался с конфликтами и в конечном итоге сделал форс пуш. Получилась такая картина - это уже похоже на правду или все еще чего-то не хватает?)

fey · 2023-02-22T21:32:57Z

@antonDedyaev по коммитам вроде норм. Но самое главное - оно работает? :_)
@dzencot поревьювишь?

antonDedyaev · 2023-02-22T21:36:51Z

Поскольку твоей мейн должен быть 1 в 1 с удаленным Хекслета (т.е. ты сперва делаешь на мейне git pull --rebase upstream main, а ребейз).
В итоге комада ребейза будет такая git rebase -i main, если я не ошибаюсь.

@antonDedyaev по коммитам вроде норм. Но самое главное - оно работает? :_)
@dzencot поревьювишь?

Локально запускал - все работает)

ashikov · 2023-02-28T08:15:56Z

@dzencot тут бы поревьювить

dzencot · 2023-03-09T19:46:44Z

По коду все ок, единственный вопрос: точно нужен стейт для этого в редаксе? Возможно тут можно обойтись хуком useState, если это состояние нужно только для этого компонента.

ssssank · 2023-03-17T06:20:21Z

@antonDedyaev @dzencot принимаем ПР или всё-таки доработаем?

antonDedyaev · 2023-03-17T07:14:26Z

@antonDedyaev @dzencot принимаем ПР или всё-таки доработаем?

Могу переделать как написал @dzencot без стейта в редаксе.

fey · 2023-03-17T08:27:42Z

@antonDedyaev сделай ишшус, в, коде поставь TODO: <ссылка на ишшус и комментарий, чтобы не забыть.
Если ты не успеешь внести исправление - это сможет сделать кто-то другой ну и можем мы понять, что нужно доработать.

antonDedyaev · 2023-03-17T08:55:29Z

@antonDedyaev сделай ишшус, в, коде поставь TODO: <ссылка на ишшус и комментарий, чтобы не забыть.
Если ты не успеешь внести исправление - это сможет сделать кто-то другой ну и можем мы понять, что нужно доработать.

Ок. А с этим ПР что в итоге - он будет принят или отклонен?
А можно мне в своем же ишшусе отписаться, что беру в работу ?)))

fey · 2023-03-17T09:14:17Z

@antonDedyaev да, будет принят, только нужно туду поставить комментом.

antonDedyaev · 2023-03-17T09:41:55Z

@antonDedyaev да, будет принят, только нужно туду поставить комментом.

Создал новый ишшус (#1526) и добавил коммент с ссылкой на него в код для этого ПР

fey requested a review from dzencot February 16, 2023 11:34

fey reviewed Feb 20, 2023

View reviewed changes

antonDedyaev added 6 commits February 22, 2023 23:16

add logic for hiding teacher solution

1475e65

add texts for the countdown screen

c58d0ed

install deps for countdown implementation

8a910ca

set the timer for 20 min and remove unnecessary code

2b4a5fe

move clock img for the countdown to another dir

a06d6d6

change clock image import path

6ff52fe

antonDedyaev force-pushed the hide-solution branch from b1ecfea to 6ff52fe Compare February 22, 2023 20:54

dzencot approved these changes Mar 9, 2023

View reviewed changes

add comment with a link to the refactoring issue

4eb0db3

fey merged commit 3399840 into Hexlet:main Mar 17, 2023

antonDedyaev deleted the hide-solution branch March 17, 2023 12:04

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix for #1348 Add a timer to the tab with the solution #1499

Fix for #1348 Add a timer to the tab with the solution #1499

antonDedyaev commented Feb 15, 2023

fey commented Feb 16, 2023

antonDedyaev commented Feb 16, 2023 •

edited

Loading

fey commented Feb 16, 2023

antonDedyaev commented Feb 16, 2023

fey commented Feb 16, 2023

fey commented Feb 16, 2023

antonDedyaev commented Feb 16, 2023

fey commented Feb 16, 2023

ashikov commented Feb 20, 2023

antonDedyaev commented Feb 20, 2023

fey Feb 20, 2023

fey commented Feb 20, 2023

ashikov commented Feb 21, 2023

antonDedyaev commented Feb 21, 2023 •

edited

Loading

fey commented Feb 21, 2023

antonDedyaev commented Feb 22, 2023

fey commented Feb 22, 2023

antonDedyaev commented Feb 22, 2023 •

edited

Loading

fey commented Feb 22, 2023

antonDedyaev commented Feb 22, 2023

ashikov commented Feb 28, 2023

dzencot commented Mar 9, 2023

ssssank commented Mar 17, 2023

antonDedyaev commented Mar 17, 2023

fey commented Mar 17, 2023

antonDedyaev commented Mar 17, 2023

fey commented Mar 17, 2023

antonDedyaev commented Mar 17, 2023 •

edited

Loading

Fix for #1348 Add a timer to the tab with the solution #1499

Fix for #1348 Add a timer to the tab with the solution #1499

Conversation

antonDedyaev commented Feb 15, 2023

fey commented Feb 16, 2023

antonDedyaev commented Feb 16, 2023 • edited Loading

fey commented Feb 16, 2023

antonDedyaev commented Feb 16, 2023

fey commented Feb 16, 2023

fey commented Feb 16, 2023

antonDedyaev commented Feb 16, 2023

fey commented Feb 16, 2023

ashikov commented Feb 20, 2023

antonDedyaev commented Feb 20, 2023

fey Feb 20, 2023

Choose a reason for hiding this comment

fey commented Feb 20, 2023

ashikov commented Feb 21, 2023

antonDedyaev commented Feb 21, 2023 • edited Loading

fey commented Feb 21, 2023

antonDedyaev commented Feb 22, 2023

fey commented Feb 22, 2023

antonDedyaev commented Feb 22, 2023 • edited Loading

fey commented Feb 22, 2023

antonDedyaev commented Feb 22, 2023

ashikov commented Feb 28, 2023

dzencot commented Mar 9, 2023

ssssank commented Mar 17, 2023

antonDedyaev commented Mar 17, 2023

fey commented Mar 17, 2023

antonDedyaev commented Mar 17, 2023

fey commented Mar 17, 2023

antonDedyaev commented Mar 17, 2023 • edited Loading

antonDedyaev commented Feb 16, 2023 •

edited

Loading

antonDedyaev commented Feb 21, 2023 •

edited

Loading

antonDedyaev commented Feb 22, 2023 •

edited

Loading

antonDedyaev commented Mar 17, 2023 •

edited

Loading