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

Added appstat resolver #862

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dimasites
Copy link

Что оно делает?

Добавлен резолвер для сбора community stat

Зачем это нужно?

Статистика позволит улучшить продукт (minishop2) на основе данных.
Собираются только:
Версия пакета, которая устанавливается или удаляется
Версия, с которой обновляется пакет
Версия MODX, PHP и язык админки.

Пара скриншотов

Установка:
2023-04-20_15-14-08

Обновление (на примере другого пакета):
photo_2023-04-20_15-21-09

Решение проверено в течение 30 дней на почти десятке пакетов. Скриншоты выкладывал в telegram-сообществе.

@dimasites
Copy link
Author

dimasites commented Apr 30, 2023

Прибрался в коммитах, убрал всё лишнее, слил все изменения в 1 вот этот коммит

А остальные коммиты чужие, что светятся в PR, это по сути слияние с мастер-веткой (т.к. я по неаккуратности на старте работ сделал ответвление без предварительного нажатия sync fork.

Следы моего другого PR тоже отсюда вычистил!

@dimasites dimasites force-pushed the feature/appstat branch 2 times, most recently from 18b3b29 to f9dbe4c Compare April 30, 2023 23:09
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.

2 participants