Skip to content

Support rich messages, topic read state, and automatic releases - #12

Merged
AbdulKus merged 2 commits into
mainfrom
feature/rich-messages-topic-read-release
Aug 9, 2026
Merged

Support rich messages, topic read state, and automatic releases#12
AbdulKus merged 2 commits into
mainfrom
feature/rich-messages-topic-read-release

Conversation

@AbdulKus

@AbdulKus AbdulKus commented Aug 9, 2026

Copy link
Copy Markdown
Owner

Заменяет автоматически закрывшийся #11; ветка и коммит сохранены.

Что изменено

  • отображение messageRichMessage: превью в чате/списке и полное содержимое во встроенном просмотрщике
  • новые блоки форматирования, custom emoji и читаемое отображение формул
  • сохранение прочтения и позиции перед переключением топика
  • автоматический GitHub Release с APK и SHA256SUMS.txt после успешной сборки main
  • английская и русская локализации

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a7ccbea4c6

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines 233 to +234
public boolean isCustomEmoji () {
return false;
return emoji != null;

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Provide media handling for custom emoji in reference tooltips

When a user opens an instant-view reference whose reference text contains a custom emoji, this now identifies the entity as media, but InstantViewController.onReferenceClick builds that tooltip with a null TextMediaListener. During text preparation, Text.processEmoji calls newOrExistingMedia, which throws IllegalStateException when the listener is null, crashing the view. Supply a media listener for this tooltip or retain a non-media fallback when no listener is available.

Useful? React with 👍 / 👎.

Comment on lines +153 to +156
release:
name: Publish GitHub Release
if: github.event_name == 'push' && github.ref == 'refs/heads/main'
needs: build

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Require a persistent signing key before publishing releases

On a main-branch push where any signing secret is absent, the build job deliberately generates a fresh ephemeral keystore, yet this unconditional main-push release job publishes that APK as the latest release. Because every such run uses a different signing identity, users who install one automatic release cannot update to the next one; gate publication on the complete signing-secret set while leaving ephemeral builds as CI artifacts only.

Useful? React with 👍 / 👎.

@AbdulKus
AbdulKus merged commit d2c7947 into main Aug 9, 2026
2 checks 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.

1 participant