Skip to content

Allow repo to be an array of repository links#3123

Open
Soniraja17 wants to merge 1 commit intomatrix-org:mainfrom
Soniraja17:feature/repo-array-support
Open

Allow repo to be an array of repository links#3123
Soniraja17 wants to merge 1 commit intomatrix-org:mainfrom
Soniraja17:feature/repo-array-support

Conversation

@Soniraja17
Copy link

@Soniraja17 Soniraja17 commented Dec 25, 2025

Description

Fixes #2125

Support multiple repository links for clients that have separate repos for different platforms (e.g., Element).

  • Update client.html template to handle both string and array formats
  • Update client_details.html macro similarly
  • Maintain backward compatibility with existing single-string repos

Related issues

Fixes #2125

Role

Individual contributor

Timeline

No specific deadline — review at your convenience

Signoff

Please sign off your individual commits or whole pull request.

Fixes matrix-org#2125

Support multiple repository links for clients that have separate
repos for different platforms (e.g., Element).

- Update client.html template to handle both string and array formats
- Update client_details.html macro similarly
- Maintain backward compatibility with existing single-string repos

Signed-off-by: Soniraja17 <your.email@example.com>
@Soniraja17 Soniraja17 requested a review from a team as a code owner December 25, 2025 18:23
{% for repo_url in page.extra.repo %}
<a href="{{ repo_url }}" class="pill">
{{ load_data(path="/assets/repository.svg") | safe }}
Source Code
Copy link
Contributor

Choose a reason for hiding this comment

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

Just having "Source Code" multiple times seems not very useful for endusers. It probably should have some kind of title. In the case of Element if for example could say something like "Android Source Code" and "iOS Source Code" but this may differ with different examples

Copy link
Contributor

@MTRNord MTRNord left a comment

Choose a reason for hiding this comment

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

We can't accept a bogus email on signoffs as used in the commit. Please follow the CONTRIBUTING.md rules on the signoff requirements.

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.

Allow having an array of repo links

2 participants