frontend
/
spa_instat_tv
1
0
Fork 0
Code Pull Requests 1 Releases Wiki Activity
Labels Milestones

fix(#658): build fqtv to diwan.insports.tv #234

Merged
andrey.dekterev merged 1 commits from in-658-fqtv-client into develop 3 years ago
Conversation 0 Commits 1 Files Changed 23
roman.rakov commented 3 years ago
Owner

https://insports.atlassian.net/browse/IN-658

https://insports.atlassian.net/browse/IN-658
roman.rakov added 1 commit 3 years ago
continuous-integration/drone/push Build is passing
Details
0e4cf7ef7b fix(#658): build fqtv to diwan.insports.tv
roman.rakov requested review from andrey.dekterev 3 years ago
roman.rakov requested review from denis.farber 3 years ago
andrey.dekterev reviewed 3 years ago
.gitignore Outdated
/build_facr
/build_lff
/build_tunisia
/build_fqtv
andrey.dekterev commented 3 years ago
Owner

замени все папки build на build*

замени все папки build на build*
roman.rakov marked this conversation as resolved
andrey.dekterev reviewed 3 years ago
src/config/clients/types.tsx Outdated
export enum ClientNames {
Brasil = 'brasil',
Facr = 'facr',
Fqtv = 'fqtv',
andrey.dekterev commented 3 years ago
Owner

добавь плз тут комменты со странами для которых эти клиенты

добавь плз тут комменты со странами для которых эти клиенты
roman.rakov marked this conversation as resolved
andrey.dekterev reviewed 3 years ago
src/features/BuyMatchPopup/components/PackageSelectionStep/index.tsx Outdated
switch (countryCode?.country_code) {
case 'BR':
andrey.dekterev commented 3 years ago
Owner

тут тоже обозначения стран лучше привязать к стране, чтобы более наглядно было и вынести в enum

тут тоже обозначения стран лучше привязать к стране, чтобы более наглядно было и вынести в enum
roman.rakov marked this conversation as resolved
andrey.dekterev reviewed 3 years ago
src/features/HeaderMobile/styled.tsx Outdated
) => {
if (['lff', 'tunisia'].includes(client.name)) {
if ([
ClientNames.Lff,
andrey.dekterev commented 3 years ago
Owner

кажется пора написать функцию

кажется пора написать функцию
roman.rakov commented 3 years ago
Poster
Owner

эту функицю ожидает рефактор

эту функицю ожидает рефактор
roman.rakov marked this conversation as resolved
andrey.dekterev reviewed 3 years ago
src/features/HeaderMobile/styled.tsx Outdated
#000000 50%),
#000000;
z-index: 10;
`
andrey.dekterev commented 3 years ago
Owner

и тут пора написать функцию

и тут пора написать функцию
roman.rakov commented 3 years ago
Poster
Owner

эту функицю ожидает рефактор

эту функицю ожидает рефактор
roman.rakov marked this conversation as resolved
andrey.dekterev reviewed 3 years ago
src/features/ProfileHeader/styled.tsx Outdated
ClientNames.Tunisia,
ClientNames.Facr,
ClientNames.Fqtv,
].includes(client.name)
andrey.dekterev commented 3 years ago
Owner

и тут пора написать функцию

и тут пора написать функцию
roman.rakov marked this conversation as resolved
andrey.dekterev reviewed 3 years ago
src/features/ProfileHeader/styled.tsx Outdated
client.name === ClientNames.Lff || client.name === ClientNames.Facr || client.name === ClientNames.Tunisia ? css`
client.name === ClientNames.Lff
|| client.name === ClientNames.Facr
|| client.name === ClientNames.Fqtv
andrey.dekterev commented 3 years ago
Owner

и тут пора написать функцию

и тут пора написать функцию
roman.rakov marked this conversation as resolved
andrey.dekterev reviewed 3 years ago
src/features/StreamPlayer/components/Controls/Components/ControlsMobile/styled.tsx Outdated
export const Controls = styled.div<FullscreenProps>`
z-index: 14;
z-index: 9;
andrey.dekterev commented 3 years ago
Owner

уверен, что это ни на что не повлияет?

уверен, что это ни на что не повлияет?
roman.rakov commented 3 years ago
Poster
Owner

тестеры не увидели ничего

тестеры не увидели ничего
roman.rakov marked this conversation as resolved
andrey.dekterev reviewed 3 years ago
src/features/Theme/config.tsx Outdated
background: '#1B1E27',
button: '#016BD8',
buttonHover: '#016BD8',
collapseTournamentBackground: 'linear-gradient(236.13deg, rgba(1, 48, 121, 0.8) -4.49%, rgba(52, 62, 73, 0.8) 98.29%), #363F49;',
andrey.dekterev commented 3 years ago
Owner

вынеси все цвета в отдельную страницу и давай брать их оттуда в конфиги, сделаем что-то вроде дизайн-системы

вынеси все цвета в отдельную страницу и давай брать их оттуда в конфиги, сделаем что-то вроде дизайн-системы
roman.rakov commented 3 years ago
Poster
Owner

оставил это без изменений

оставил это без изменений
roman.rakov marked this conversation as resolved
denis.farber approved these changes 3 years ago
roman.rakov force-pushed in-658-fqtv-client from 0e4cf7ef7b to 767d7794c3 3 years ago
andrey.dekterev merged commit 767d7794c3 into develop 3 years ago
andrey.dekterev deleted branch in-658-fqtv-client 3 years ago

Reviewers

andrey.dekterev was requested for review 3 years ago
denis.farber approved these changes 3 years ago
continuous-integration/drone/push Build is passing
Details
The pull request has been merged as 767d7794c3.
Sign in to join this conversation.
Reviewers
Request review
No reviewers
andrey.dekterev
denis.farber
Labels
Apply labels
Clear labels
No items
No Label
Milestone
Set milestone
Clear milestone
No items
No Milestone
Assignees
Assign users
Clear assignees
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: frontend/spa_instat_tv#234
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'in-658-fqtv-client'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes