Skip to content

Conversation

aha999
Copy link

@aha999 aha999 commented Nov 8, 2021

I have improved readme by adding new content like new high resolution download icons, gif next to the screenshots, donation info, contribution info etc. generally it looks and functions better. It was made because of the issue #777

You can see it in my fork: https://github.com/aha999/Transportr

@cla-bot
Copy link

cla-bot bot commented Nov 8, 2021

Thank you for your pull request and welcome to our community! We require contributors to sign our Contributor License Agreement, and we don't seem to have the user @aha999 on file. In order for your code to get reviewed and merged, please explicitly state that you accept the agreement. Alternatively, you can add a commit that adds yourself to https://github.com/grote/Transportr/blob/master/.clabot

@aha999 aha999 mentioned this pull request Nov 8, 2021
@aha999
Copy link
Author

aha999 commented Nov 9, 2021

I accept the agreement

@Altonss
Copy link
Collaborator

Altonss commented May 23, 2023

This PR conflicts with the current master branch, so for a merge this would need to be resolved.
Also why make that much changes in the readme? To me it would be sufficient to replace the F-Droid and PlayStore badges, and maybe the move the Transportr logo to the top.

@ialokim
Copy link
Collaborator

ialokim commented May 23, 2023

Also, with these changes every load of the README issues a lot of third-party requests. Not sure if we'd want that.

@Altonss
Copy link
Collaborator

Altonss commented May 23, 2023

Also, with these changes every load of the README issues a lot of third-party requests. Not sure if we'd want that.

Are those really generating third-party requests? Are they not loaded through github servers?

@ialokim
Copy link
Collaborator

ialokim commented May 23, 2023

Also, with these changes every load of the README issues a lot of third-party requests. Not sure if we'd want that.

Are those really generating third-party requests? Are they not loaded through github servers?

At least when looking at the (rendered) Markdown locally.

@aha999
Copy link
Author

aha999 commented Jun 4, 2023

@Altonss i make that many merges, because i don't change everything in one try, so sometimes i save and continue later.
Which 3'rd party requests are we talking about?

@ialokim
Copy link
Collaborator

ialokim commented Aug 13, 2023

I just double-checked and after the changes, the README only loads images additionally from shields.io. I think that's actually fine with me.


The public transport companion that respects your privacy and your freedom.
Transportr is a non-profit app developed by people around the world to make using public transport as easy as possible wherever you are.

[![Transportr Logo](/app/src/main/res/mipmap-xhdpi/ic_launcher.png)](https://transportr.app)
[![Build Status](https://travis-ci.org/grote/Transportr.svg?branch=master)](https://travis-ci.org/grote/Transportr)
Copy link
Collaborator

Choose a reason for hiding this comment

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

This one doesn't work anymore. Anyone knows how to get the build status from the Github CI instead?

Choose a reason for hiding this comment

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

Wouldn't the current one used in the README.md (Build and test)
[![Build and test](https://github.com/grote/Transportr/actions/workflows/build.yml/badge.svg)](https://github.com/grote/Transportr/actions/workflows/build.yml)
work just fine?


Please **[visit the website](https://transportr.app)** for more information!
* Pre-releases and beta versions for advanced users are available via [a special F-Droid repository](http://grobox.de/fdroid/).
Copy link
Collaborator

Choose a reason for hiding this comment

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

I wouldn't add a bullet point here.


The upstream repository is at: https://github.com/grote/Transportr
[<img src="https://user-images.githubusercontent.com/50620416/140822064-86778b46-f701-40b0-a9bd-fc10d1397f0f.gif" height="444">](https://transportr.app/img/transportr.webm)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Where does this come from? Is it a gif or a webm video? In either case, it should be added to the repo if we want to show it in the README.

* Share Transportr on social media or show your friends and family that it exists and how to use it.
* If you find any issues with this app, please report them at [the issue tracker](https://github.com/grote/Transportr/issues).
* Rate it in [Google Play](https://play.google.com/store/apps/details?id=de.grobox.liberario) and if you have a little more time also write a short review about it.
* Add more supported regions by contributing to the [Transport-Enabler](https://github.com/schildbach/public-transport-enabler/) project, which is used by Transportr. More methods are explained on the website.
Copy link
Collaborator

Choose a reason for hiding this comment

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

The library is called Public Transport Enabler (PTE)

</tr>
<tr>
<td align="center"><a href="bitcoin:12JaQp8zfqRb83JfSwVjH4rZWsZnWRPoyG"><img src="https://transportr.app/img/bitcoin-qr.png" alt="bitcoin:12JaQp8zfqRb83JfSwVjH4rZWsZnWRPoyG" width="128px" height="128px"></a></td>
<td align="center">Bitcoin</a></td>
Copy link
Collaborator

Choose a reason for hiding this comment

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

@grote Do you still want to accept donations via Bitcoin?

Copy link
Owner

Choose a reason for hiding this comment

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

I don't care either way. The original address should still work and I have still the key, but distributing it to other people is more difficult, so LiberaPay is probably better.

@Altonss Altonss mentioned this pull request Nov 1, 2024
@Altonss
Copy link
Collaborator

Altonss commented Nov 1, 2024

The badges have been fixed in #955, so I'll close this PR for now as the scope of this PR seems too big. Feel free to reopen maybe smaller PRs easier to review :)

@Altonss Altonss closed this Nov 1, 2024
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.

5 participants