element-hq / element-hq/element-android

Move RoomSummaryConstants to MatrixConfiguration

Open
#4,502 0 comments 0 reactions 0 assignees View on GitHub
matrix-sdk
Dominant language
Kotlin
Stars
3.7k
Forks
917
PR merge metrics
No merged PRs in 30d

Description

Clients of the SDK should be able to decide what can be considered as a previewable events (used for room list for instance).

Documentation must mention that changing this value may lead to unexpected behavior.

Currently this is hard-coded here: https://github.com/vector-im/element-android/blob/main/matrix-sdk-android/src/main/java/org/matrix/android/sdk/api/session/room/summary/RoomSummaryConstants.kt#L26

The configuration can have this default value, or if null use this default value, not sure what is the best approach here.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.