Skip to content

Conversation

@gongchandang49
Copy link

Description

Summary
Appended all public homeservers to the current list, and made it scrollable because with current styling it couldn't be displayed well.

Motivation
Today's downtime of matrix.org homesever, that forced me to federate, and I wanted to keep using Cinny for it.

Code context
Integrates correctly with the current codebase, the only changes are some lines appended to config.json and a minor style edit in ServerPicker.tsx. A preview of this functionality can be found on my fork: https://cinnyapp.pages.dev

Type of change

  • New feature (non-breaking change which adds functionality)

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code

@github-actions
Copy link

github-actions bot commented Sep 3, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@gongchandang49
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

ajbura added a commit to cinnyapp/cla that referenced this pull request Sep 3, 2025
Copy link
Collaborator

@kfiven kfiven left a comment

Choose a reason for hiding this comment

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

To be added to this list, a server must:

  1. have open registration
  2. their registration method should be supported in Cinny
  3. should be reputable
  4. login/register flow needs to be tested before adding.

A quick check shows few of server offline, closed registration.

@gongchandang49
Copy link
Author

  1. should be reputable

How to objectively measure this?

  1. login/register flow needs to be tested before adding.

So I need to test the login and registration flow for every single one of these servers, in order for them to get added?

@kfiven
Copy link
Collaborator

kfiven commented Sep 15, 2025

To measure server reputation, I usually check how long they are operating, downtime, if they are blocked or not in community moderation list or matrix.org block list.

Yes we need to test the flows as it will feel broken if these features are not working because some server only allow registration from their choice of client.

@gongchandang49
Copy link
Author

To measure server reputation, I usually check how long they are operating, downtime, if they are blocked or not in community moderation list or matrix.org block list.

Great! In that case all of them fulfil those requirements, see here https://servers.joinmatrix.org/operators.html (those 3 are requirements manually checked by the list maintainer).
Regarding the ones that are down and/or have closed public registration, I can filter them out easily.

Yes we need to test the flows as it will feel broken if these features are not working because some server only allow registration from their choice of client.

Ok, in that case I will make a dummy account on all those servers and test registration flow. Is it ok that I just confirm the working ones, or do I need to screen record all of the tests? This can be quite time-consuming, honestly.

@kfiven
Copy link
Collaborator

kfiven commented Sep 19, 2025

Yeah, just testing is enough. No need to record.

Comment on lines +118 to +119
overflowY: 'auto',
WebkitOverflowScrolling: 'touch',
Copy link
Member

Choose a reason for hiding this comment

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

We should use Scroll component from folds for scrollable container. Other "menus" can be looks for ref.

"sans-nuage.fr",
"libreon.fr",
"transgirl.cafe",
"deuxfleurs.fr"
Copy link
Member

Choose a reason for hiding this comment

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

servers should be sorted A-Z? @kfiven

Copy link
Collaborator

Choose a reason for hiding this comment

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

yes

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.

3 participants