organicmaps/organicmaps-tmp

Archived

WIP: [android][ci] Import metadata into repo #982

Closed

rtsisyk wants to merge 1 commit from rt-android-metadata into master

rtsisyk commented

2021-08-02 10:05:33 +00:00

Owner

Signed-off-by: Roman Tsisyk roman@tsisyk.com

Signed-off-by: Roman Tsisyk <roman@tsisyk.com>

rtsisyk reviewed 2021-08-02 10:06:23 +00:00

android/src/main/play/listings/en-US/full-description.txt

					
				@ -0,0 +1,38 @@

				• Detailed offline maps with places that don't exist on other maps

rtsisyk commented

2021-08-02 10:06:23 +00:00

Author

Owner

@biodranik давай только скрины вынесу в отдельную репу, а метаданные оставлю здесь? Метаданные не хочется отрывать от кода. Скрины тоже не хочется, но их реже меняем и они жирные.

@biodranik давай только скрины вынесу в отдельную репу, а метаданные оставлю здесь? Метаданные не хочется отрывать от кода. Скрины тоже не хочется, но их реже меняем и они жирные.

rtsisyk reviewed 2021-08-02 10:08:00 +00:00

.github/workflows/android-release.yaml

					
				@ -90,0 +70,4 @@

				        shell: bash

				        working-directory: android

				        run: |

				          ./gradlew assembleWebRelease

rtsisyk commented

2021-08-02 10:08:00 +00:00

Author

Owner

Могу листинги в гугле отдельно обновлять. Но в эппле всё равно это так не работает, можно только с заливкой. Поэтому решил здесь вместо с заливкой bundle сразу апдейт меты делать. Апдейт меты поменяет её практически сразу же, раньше чем пройдет ревью аппы.

Могу листинги в гугле отдельно обновлять. Но в эппле всё равно это так не работает, можно только с заливкой. Поэтому решил здесь вместо с заливкой bundle сразу апдейт меты делать. Апдейт меты поменяет её практически сразу же, раньше чем пройдет ревью аппы.

biodranik (Migrated from github.com) reviewed 2021-08-02 12:17:35 +00:00

.github/workflows/android-release.yaml

					
				@ -90,0 +70,4 @@

				        shell: bash

				        working-directory: android

				        run: |

				          ./gradlew assembleWebRelease

biodranik (Migrated from github.com) commented

2021-08-02 12:17:35 +00:00

Мета включает what's new? Или это всё остальное, кроме?

Мета включает what's new? Или это всё остальное, кроме?

biodranik (Migrated from github.com) reviewed 2021-08-02 12:22:04 +00:00

android/src/main/play/listings/en-US/full-description.txt

					
				@ -0,0 +1,38 @@

				• Detailed offline maps with places that don't exist on other maps

biodranik (Migrated from github.com) commented

2021-08-02 12:22:04 +00:00

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

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

biodranik (Migrated from github.com) reviewed 2021-08-02 12:25:59 +00:00

android/src/main/play/listings/en-US/full-description.txt

					
				@ -0,0 +5,4 @@

				• Fast offline search on the map and bookmarks

				• Dark mode to protect your eyes

				Organic Maps is <b>pure and organic, made with love</b>:

biodranik (Migrated from github.com) commented

2021-08-02 12:22:53 +00:00

А нельзя исходные тексты в markdown?

А нельзя исходные тексты в markdown?

android/src/main/play/listings/en-US/full-description.txt

					
				@ -0,0 +11,4 @@

				• Saves your battery

				• No unexpected mobile data charges

				Organic Map is <b>free from trackers and other bad stuff</b>:

biodranik (Migrated from github.com) commented

2021-08-02 12:23:02 +00:00

Maps app

Maps app

tools/unix/generate_changelog.sh

					
				@ -0,0 +2,4 @@

				set -euo pipefail

				EXCLUDE="\[tools\]|\[string\]\[generator\]"

biodranik (Migrated from github.com) commented

2021-08-02 12:24:38 +00:00

Фильтр надо допилить. Стринги лучше оставить, но без регенерации.

Фильтр надо допилить. Стринги лучше оставить, но без регенерации.

tools/unix/generate_changelog.sh

					
				@ -0,0 +6,4 @@

				case "${1:-default}" in

				  firebase)

				    LAST_TAG=$(git describe --tags --abbrev=0)

biodranik (Migrated from github.com) commented

2021-08-02 12:25:45 +00:00

Надо не последний тег, а последний ведро или последний айос тег.

Надо не последний тег, а последний ведро или последний айос тег.

rtsisyk reviewed 2021-08-04 09:04:07 +00:00

.github/workflows/android-release.yaml

					
				@ -90,0 +70,4 @@

				        shell: bash

				        working-directory: android

				        run: |

				          ./gradlew assembleWebRelease

rtsisyk commented

2021-08-04 09:02:16 +00:00

Author

Owner

What's new:

Должно лежать в src/main/play/release-notes/en-US/{alpha,beta,default).txt (в этом PR еще пока нет).
Обновляется при заливке APK в стор.

Metadata:

Лежит в android/src/main/play/listings/
Можно обновлять в любой момент, к релизам никак не привязано.

Получается, что what's new по идее надо бы в основной репе хранить, так как оно явно привязано к релизам. Обновление метаданных стора не привязано и можно хранить отдельно. При этом пихать это в разные репы не очень хочется. Наверное, было бы правильным, чтобы whatsinew лежало по buildnumber.txt. Гляну как fastlane делает, правда не хочется на это УГ переходить.

What's new: - Должно лежать в src/main/play/release-notes/en-US/{alpha,beta,default).txt (в этом PR еще пока нет). - Обновляется при заливке APK в стор. Metadata: - Лежит в android/src/main/play/listings/ - Можно обновлять в любой момент, к релизам никак не привязано. Получается, что what's new по идее надо бы в основной репе хранить, так как оно явно привязано к релизам. Обновление метаданных стора не привязано и можно хранить отдельно. При этом пихать это в разные репы не очень хочется. Наверное, было бы правильным, чтобы whatsinew лежало по buildnumber.txt. Гляну как fastlane делает, правда не хочется на это УГ переходить.

android/src/main/play/listings/en-US/full-description.txt

					
				@ -0,0 +1,38 @@

				• Detailed offline maps with places that don't exist on other maps

rtsisyk commented

2021-08-04 08:57:34 +00:00

Author

Owner

С правами проблем нет, мы работаем через pull requests. Даже с правами на запись в эту репу основной бранч запротекчен. Так что этот аргумент давай исключим из обсуждения.

С правами проблем нет, мы работаем через pull requests. Даже с правами на запись в эту репу основной бранч запротекчен. Так что этот аргумент давай исключим из обсуждения.

android/src/main/play/listings/en-US/full-description.txt

					
				@ -0,0 +5,4 @@

				• Fast offline search on the map and bookmarks

				• Dark mode to protect your eyes

				Organic Maps is <b>pure and organic, made with love</b>:

rtsisyk commented

2021-08-04 08:56:26 +00:00

Author

Owner

У гугла весьма ограниченный HTML в описании. Давай не будем усложнять себе жизнь.

У гугла весьма ограниченный HTML в описании. Давай не будем усложнять себе жизнь.

biodranik commented

2021-08-04 14:42:45 +00:00

(Migrated from github.com)

Как я уже писал, мне не нравится:

Разнесение скриншотов от метадаты.
Раздувание основной репы ещё и скриншотами.
Отсутствие контроля максимальной длины символов в редактируемых руками текстовых файлах, что неизбежно будет приводить к ошибкам аплоада метадаты и трате времени.

По первым двум пунктам напрашивается отдельный репо, что логично. What's new всё равно нужно аккуратно лепить руками, не вижу ничего страшного, что в отдельном репо это будет разложено по релизам или затегано как нужно.

По третьему пункту надо думать, либо автовалидатор для PR/пушей/коммитов, либо, что более предпочтительно, какая-то удобная веб морда, т.к. всё равно работа сводится к текстовому редактору.

Как я уже писал, мне не нравится: 1. Разнесение скриншотов от метадаты. 2. Раздувание основной репы ещё и скриншотами. 3. Отсутствие контроля максимальной длины символов в редактируемых руками текстовых файлах, что неизбежно будет приводить к ошибкам аплоада метадаты и трате времени. По первым двум пунктам напрашивается отдельный репо, что логично. What's new всё равно нужно аккуратно лепить руками, не вижу ничего страшного, что в отдельном репо это будет разложено по релизам или затегано как нужно. По третьему пункту надо думать, либо автовалидатор для PR/пушей/коммитов, либо, что более предпочтительно, какая-то удобная веб морда, т.к. всё равно работа сводится к текстовому редактору.

rtsisyk commented

2021-08-07 19:13:54 +00:00

Author

Owner

Replaced by #1035

Replaced by #1035

This repo is archived. You cannot comment on pull requests.

No reviewers

No labels

Android Automotive (AAOS)

Android Automotive (AAOS)

Battery and Performance

Battery and Performance

Bookmarks and Tracks

Bookmarks and Tracks

External Map Datasets

External Map Datasets

Frequently User Reported

Frequently User Reported

Good first issue

Good first issue

Linux packaging

Linux packaging

NLnet 2024-06-281

NLnet 2024-06-281

No Feature Parity

No Feature Parity

Public Transport

Public Transport

Track Recording

Track Recording

Walk Navigation

Walk Navigation

No milestone

No project

No assignees

2 participants

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: organicmaps/organicmaps-tmp#982

No description provided.