Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated the cloud providers. #63

Merged
merged 4 commits into from
Oct 11, 2023
Merged

Updated the cloud providers. #63

merged 4 commits into from
Oct 11, 2023

Conversation

vdbhb59
Copy link
Contributor

@vdbhb59 vdbhb59 commented Sep 25, 2023

Removed:

  1. vern instance as is dead for sometime now, and the whole vern domain itself is dead.
  2. Facil services, as they no longer provide for FreshRSS instance.
  3. Zaclys instance as they too no longer provide RSS instance.

Updated the RSS direct instance details and URLs, so that they take users to the correct place instead of having to find, navigate, locate and what not in the pages of the domains they are hosted in. Makes it much simpler and directs to path of the instance itself.

Removed:
1. vern instance as is dead for sometime now, and the whole vern domain itself is dead.
2. Facil services, as they no longer provide for FreshRSS instance.
3. Zaclys instance as they too no longer provide RSS instance.

Updated the RSS direct instance details and URLs, so that they take users to the correct place instead of having to find, navigate, locate and what not in the pages of the domains they are hosted in. Makes it much simpler and directs to path of the site itself.
pages/cloud-providers.html Outdated Show resolved Hide resolved
Copy link
Member

@marienfressinaud marienfressinaud left a comment

Choose a reason for hiding this comment

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

~vern instance seems dead indeed, but the other providers still proposes FreshRSS as a service.

pages/cloud-providers.html Outdated Show resolved Hide resolved
pages/cloud-providers.html Outdated Show resolved Hide resolved
pages/cloud-providers.html Show resolved Hide resolved
pages/cloud-providers.html Show resolved Hide resolved
Reverted and also ensured landing pages are available.
As per PR #63
@vdbhb59
Copy link
Contributor Author

vdbhb59 commented Sep 30, 2023

All changes done as requested, and Landing pages are also available along with the main instance pages. Kindly check and merge.

@vdbhb59
Copy link
Contributor Author

vdbhb59 commented Oct 8, 2023

Unless anything pending for me, may I request this PR be merged?

@Alkarex
Copy link
Member

Alkarex commented Oct 8, 2023

Ping @marienfressinaud

It should be easier to maintain as the rows are automatically handled by
the grid layout (= no need for an additional container per row).
@marienfressinaud
Copy link
Member

I've simplified and made more coherent the links. I prefer to have the links at the same place in the cards (at the bottom), and I made the FreshRSS links a bit smaller.

I've also changed the layout of the cloud providers to grid instead of flexbox in order to decrease the risk of breaking the layout (it happened in your PR, but you're not the first).

How it looks now:

The list of FreshRSS cloud providers. The links to the FreshRSS servers are next to the links to the cloud providers.

@vdbhb59
Copy link
Contributor Author

vdbhb59 commented Oct 11, 2023

I've simplified and made more coherent the links. I prefer to have the links at the same place in the cards (at the bottom), and I made the FreshRSS links a bit smaller.

I've also changed the layout of the cloud providers to grid instead of flexbox in order to decrease the risk of breaking the layout (it happened in your PR, but you're not the first).

How it looks now:

The list of FreshRSS cloud providers. The links to the FreshRSS servers are next to the links to the cloud providers.

Thanks. Do I need to do anything further? Will you be merging this PR please.

@marienfressinaud
Copy link
Member

If it's fine to the others, we can merge! :)

@Alkarex Alkarex merged commit a0edeba into FreshRSS:main Oct 11, 2023
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.

4 participants