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

[#254] Add workspace name on Integration page (with test) #298

Merged
merged 4 commits into from
Aug 26, 2024

Conversation

SibirBear
Copy link
Contributor

@SibirBear SibirBear commented Aug 14, 2024

The workspace name is displayed correctly on the Integration page. One of the tests for checking the name of the space on the integration page has also been adjusted.

--
На странице integration корректно отображается имя рабочего пространства. Также скорретирован один из тестов для проверки имени пространства на странице "Интеграция".
issue254

@SibirBear SibirBear changed the title #254 fixes bug & add test for it [#254] fixes bug & add test for it Aug 14, 2024
@SibirBear SibirBear changed the title [#254] fixes bug & add test for it [#254] Add workspaces name on Integration page (with test) Aug 14, 2024
@SibirBear SibirBear changed the title [#254] Add workspaces name on Integration page (with test) [#254] Add workspace name on Integration page (with test) Aug 14, 2024
@fey fey linked an issue Aug 15, 2024 that may be closed by this pull request
@fey fey requested a review from Malcom1986 August 20, 2024 12:54
@Malcom1986
Copy link
Collaborator

Can you please deploy the application so that we can check the work?

@SibirBear
Copy link
Contributor Author

@Malcom1986, hi!
Ok, here's the link https://hexlet-correction-kk2w.onrender.com

@Malcom1986
Copy link
Collaborator

Хм, в задеплоенной демке не вижу имени репозитория на странице интеграции, проверьте пожалуйста

@Malcom1986
Copy link
Collaborator

Плюс нужно еще работу с Optional поправить, избавится от проверки

@SibirBear
Copy link
Contributor Author

@Malcom1986 передеплоил, наверное другое ветку в прошлый раз дал https://hexlet-correction-25ne.onrender.com

А что конкретно имеется ввиду под "работу с Optional поправить"? Не совсем понимаю о чем речь, потому даже нет мыслей про то, что и где поправить :)

@Malcom1986
Copy link
Collaborator

Написал в ревью под кодом. Метод orElse такая изящная замена проверки Optional на пустоту и последующего получения значения

@SibirBear
Copy link
Contributor Author

SibirBear commented Aug 23, 2024

Написал в ревью под кодом. Метод orElse такая изящная замена проверки Optional на пустоту и последующего получения значения

@Malcom1986 , прошу меня извинить за глупость, но я вупор не вижу никакого окммента под кодом) Не могли бы Вы объяснить как и где это посмотреть?

@Malcom1986
Copy link
Collaborator

Malcom1986 commented Aug 23, 2024

Хм, а если пролистать немного вверх комменты, вы видите процесс ревью? Там прям участок кода и комментарий к нему. Или можно еще изменения посмотреть. Там тоже видны эти комментарии
Screenshot from 2024-08-23 15-35-32

@SibirBear
Copy link
Contributor Author

SibirBear commented Aug 25, 2024

@Malcom1986 нет, не вижу такие комменты, наверное дело в том, что ревью не закрыто?
https://stackoverflow.com/questions/10248666/how-to-view-line-comments-in-github

--
image

@Malcom1986 Malcom1986 merged commit 1381683 into Hexlet:main Aug 26, 2024
1 check passed
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.

Bug: the name of the workspace on the integration page is missing
2 participants