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.

Sign up for GitHub

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

Jump to bottom

[Feature] Сделаем VKUI Great Again #859

Closed
mahnunchik opened this issue Jul 8, 2020 · 21 comments
Closed

[Feature] Сделаем VKUI Great Again #859

mahnunchik opened this issue Jul 8, 2020 · 21 comments
Labels
type:feature

Comments

@mahnunchik
Copy link

Я понимаю, что VKUI это один из первых опенсурсных продуктов компании, которые разрабатываются внутри компании для использования как внутренними, так и внешними разработчиками приложений VK Mini Apps. Библиотека заметно ускоряет и упрощает разработку интерфейса, но подход к разработке и обратной связи заставляет желать лучшего.

Хотелось бы обратить внимание менеджмента на следующие моменты:

  1. Прозрачность работы с репортами — задачи висят по году без какого либо ответа. Для разработчиков приложений VK Mini Apps важно понимать будет ли исправлена проблема и если будет то ориентировочные сроки. В настоящее время складывается следующая ситуация: тестеры и пользователи ждут ответа от разработчика, разработчик ждёт ответа от команды VKUI и всё залипает.
  2. Связь между продуктами — я понимаю, что VKUI, vk-bridge и биндинги в платформах — это разные продукты в терминах VK, но для внешнего разработчика VK Mini Apps они связаны до неразличимости. В настоящее время складывается ситуация, когда репорт "футболят" между продуктами и не найти крайних.
  3. Доступ команды VKUI/vk-bridge к баг трекеру продуктов/приложений на ревью. Текущие репорты тестеров невозможно напрямую передать команде VKUI/vk-bridge, приходится копировать между баг трекером и GitHub-ом.
  4. Решения статуса баг трекеров продуктов VK Mini Apps, iOS, Android, Web, m.vk.com трекеры (репорты связанные с интеграцией VK Mini Apps и платформы) и VK API — репорты просматриваются и комментируются ещё хуже, чем на GitHub, а также присутствует вышеперечисленная проблема доступа когда невозможно зарепортить в "нужный" продукт.
@mahnunchik
Copy link
Author

mahnunchik commented Jul 8, 2020 •
edited
Loading

Прозрачность работы с репортами

Мои репорты висят с марта без ответа, соответствующие репорты тестеров тоже зависли:

* #596
* #597
* #598
* #599
* #600
* #602
* #603
* #605
* #619
* #620
* #624

Варианты решения:

  • Выделить специального человека на задачу разгребания и проставления статуса
  • И/или выделить время на задачу разгребания (например:среда баг трекера)

Связь между продуктами

По хорошему бага должна быть зарепорчена в iOS продукт https://vk.com/bug154507 так как проблема на стороне биндинга vk-bridge на iOS, но за пол года мне так никто и не открыл доступ к https://vk.com/bugs?act=product&id=20

Варианты решения:

  • Переработать подход к репорту и доступу к тикетам для все связки VK Mini Apps (VKUI, vk-bridge и биндинги на платформах)
  • Выделить человека/время на "переправление" репортов в правильные места
  • Написать документацию, что куда правильно репортить, чтобы проблема быстрее дошла до нужного отдела.

@ArthurStam
Copy link
Contributor

ArthurStam commented Jul 8, 2020 •
edited
Loading

  1. Привет. У нас действительно есть трудности с разгребанием проблем, особенно сейчас, когда идет активная разработка следующей мажорной версии. Не так давно мы реорганизовали работу над тикетами и вроде наметилась положительная динамика решения проблем. Понимаем вашу боль.

  2. VKUI, VK Bridge и ряд пакетов для VK Mini Apps — это действительно разные продукты для разработчиков. Тут проблемы не вижу. Разработчики должны понимать, что это независимые модули, у каждого из которых есть свой issue-трекер. Если мы видим, что "проблема не на нашей стороне", то максимум, что мы можем сделать — это направить человека в другой треккер, чат или группу.

  3. Нам комфортнее иметь один публичный канал для тикетов — GitHub. Как я вижу, багтрекер — это место для продуктовых репортов. Если выясняется, что проблема на стороне библиотеки, то необходимо завести тикет на GitHub с полным описанием того, что случилось. Недавно мы завели шаблоны для создания тикетов.
    https://github.com/VKCOM/VKUI/issues/new/choose

  4. Ответил в пункте 3.

@mahnunchik
Copy link
Author

  1. Привет! Отлично, надеюсь на положительные изменения и то, что очередь дойдёт и до моих тикетов 😉

  2. Разработчики должны понимать, что это независимые модули, у каждого из которых есть свой issue-трекер.

Считаю, что это хорошая тема для поста сюда https://vk.com/vkappsdev или даже раздела в документацию. Я то уже в курсе, но новичкам может быть полезно.

3 и 4. Аналогично, удобнее работать с единой точкой входа, в частности https://github.com/VKCOM/VKUI/issues/new/choose - шаблоны круто!

Но предлагаю разобрать случай с тикетом https://vk.com/bug154507 куда его правильно репортить?

  1. Ясно понятно, что не VKUI проблема - интерфейс системный.
  2. Можно зарепортить в https://github.com/VKCOM/vk-bridge или в https://vk.com/bugs?product=76
  3. Если ешё подебажить, то можно понять, что только на iOS проблема, то есть правильнее репортить в iOS https://vk.com/bugs?act=product&id=21

Но у большинства нет доступа к продуктам.

Как быть?

@ArthurStam
Copy link
Contributor

У меня тоже нет доступа к продуктам. Я занимаюсь разработкой данной библиотеки и хочу, чтобы мне не приходилось бегать по ссылкам и понимать, что произошло у разраба. Алгоритм заведения тикета должен быть таким (и только таким):

  1. Что-то не работает.
  2. Понимаешь, что проблема в VKUI.
  3. Создаешь тикет, идя по шаблону https://github.com/VKCOM/VKUI/issues/new/choose.
  4. Ждёшь нашей реакции.

Ссылки на другие трекеры, отхождения от шаблона — всё это замедляет работу над тикетом.

@danyadev
Copy link
Contributor

danyadev commented Jul 8, 2020

Думаю, редактирование всех тикетов @mahnunchik с добавлением напрямую в тикет шагов воспроизведения и скриншота или видео с результатом поможет разработчикам в дальнейшем оценить каждый баг, а так же даст возможность сторонним разработчикам поработать над ним не имея дел с баг трекером вк (я, к примеру, вовсе оттуда исключен)

@eolme
Copy link
Contributor

eolme commented Jul 8, 2020

vk-bridge - это тема отдельная, но очень сильно касается VKUI, например, на Android инсеты не работают и весь контент при action_bar_color: none залазит под статус и навбар, и приходится, мягко говоря, огромные костыли писать, чтобы вообще хоть как-то это исправить.

@mainsmirnov
Copy link
Contributor

@eolme этой проблеме год и о ней знают, увы фиксить не хотят.

Вообще не понимаю, второй пункт тут не вписывается. Логично, что библиотеки разные, так бы их совместили.
Все проблемы с VK Mini Apps на любой платформе можно репортить сюда: vk.com/bugs?act=product&id=76

@eolme
Copy link
Contributor

eolme commented Jul 8, 2020

Еще один вопрос по старым платформам (a.k.a Android 4 = Chrome 27-33, iOS 9), которые обговариваются как поддерживаемые, по факту оными не являются (css-variables). И, собственно, никаких альтернатив то и нет, можно конечно SASS (SCSS) или LESS с двойным переопределением для каждой темы, либо Styled Components с темами, но тоже - это огромная работа.

@fedorov-xyz
Copy link
Contributor

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

@eolme
Copy link
Contributor

eolme commented Jul 8, 2020

Хорошо, бог с ними, с темами, а что делать с m.vk.com? Я имею ввиду старые/вендорные браузеры, которым показывается вот эта страница, на которым прямо и говорится, что воспользуйтесь мобильной версией сайта. И добро пожаловать IE, Windows Phone, Opera Mini и т.п.

@mahnunchik
Copy link
Author

@ArthurStam @fedorov-xyz если бага явно VKUI то алгоритм понятен #859 (comment)

Что делать если проблема в vk-bridge или вообще непонятно где?

Как уже говорилось выше, трекер https://vk.com/bugs?act=product&id=76 очень плохо просматривается.

@mahnunchik
Copy link
Author

бля.... все ссылки на скринкасты с багами в моих тикетах протухли... нет доступа даже у меня. Проклятые продукты!

@fedorov-xyz
Copy link
Contributor

Проблема в vk-brigde – слишком размытое определение. vk-brigde – просто JS-библиотека. Скорее всего корень проблемы кроется где-то в нативном клиенте – куда и нужно репортить. Если проблема в самой библиотеке, то на GitHub – https://github.com/VKCOM/vk-bridge.

@mahnunchik
Copy link
Author

@fedorov-xyz к продуктам нативных клиентов почти ни у кого нет доступа, как уже писали выше.

До iOS продоукта мне не выдают доступ уже год.

Так куда репортить проблема которая в vk-brigde и/или нативном клиенте при условии, что ни у кого нет доступа к продуктам? Нет возможности зарепортить - нет проблем, увы не выход.

@ArthurStam
Copy link
Contributor

Хорошо, бог с ними, с темами, а что делать с m.vk.com? Я имею ввиду старые/вендорные браузеры, которым показывается вот эта страница, на которым прямо и говорится, что воспользуйтесь мобильной версией сайта. И добро пожаловать IE, Windows Phone, Opera Mini и т.п.

А какое отношение это имеет к текущему тикету?

@ArthurStam
Copy link
Contributor

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

@ArthurStam
Copy link
Contributor

ArthurStam commented Jul 8, 2020 •
edited
Loading

Я не понимаю, зачем мы тут говорим про нативные клиенты и продукты багтрекера.
Если есть проблема на стыке vkui и клиента, то репорт стоит слать в этот репозиторий, если на стыке vk-bridge и клиент — в репозиторий vk-bridge.
Если вы не понимаете, на чьей стороне проблема, то делайте догадки и шлите туда, куда считаете нужным. Мы перенаправим тикет, если ваша догадка окажется неверной.

@mahnunchik
Copy link
Author

в репозиторий vk-bridge.

В репозиторий https://github.com/VKCOM/vk-bridge или в Продукт https://vk.com/bugs?act=product&id=76

И зачем два места куда можно зарепортить?

Это противоречит, тому что написал @fedorov-xyz тут #859 (comment)

Скорее всего корень проблемы кроется где-то в нативном клиенте – куда и нужно репортить

Собственно, хотелось бы понять воркфлоу с проблемами на стыке vk-bridge и клиента: #859 (comment)

@ArthurStam
Copy link
Contributor

И зачем два места куда можно зарепортить?

Багтрекер — это место для продуктовых репортов, Github — для репортов по конкретным технлогиям (vkui, vk-bridge, icons)

Собственно, хотелось бы понять воркфлоу с проблемами на стыке vk-bridge и клиента

Если есть проблема на стыке vkui и клиента, то репорт стоит слать в этот репозиторий, если на стыке vk-bridge и клиента — в репозиторий vk-bridge.

@mahnunchik
Copy link
Author

@ArthurStam спасибо за разъяснение.

Передайте, тогда, пожалуйста, поддержке, которая отвечает за ревью приложений VK Mini Apps, чтобы НЕ отправляли в продуктовые трекеры.

@ArthurStam
Copy link
Contributor

Написал тебе на почту

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type:feature
Projects
VKUI
Archived in project
Milestone
No milestone
Development

No branches or pull requests
6 participants
@mahnunchik @ArthurStam @eolme @fedorov-xyz @danyadev @mainsmirnov