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

Update urls in "about" to avoid redirects #984

Merged
merged 2 commits into from
Jul 24, 2017

Conversation

duramato
Copy link
Contributor

No description provided.

about.pas Outdated
@@ -212,7 +212,7 @@ procedure TCheckVersionThread.Execute;
procedure TAboutForm.imgSynapseClick(Sender: TObject);
begin
AppBusy;
OpenURL('http://synapse.ararat.cz');
OpenURL('http://synapse.ararat.cz/doku.php');
Copy link
Member

Choose a reason for hiding this comment

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

It's a 302 redirection not 301, I prefer not to touch it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changed it back @PeterDaveHello

@@ -251,7 +251,7 @@ procedure TAboutForm.imgDonateClick(Sender: TObject);
procedure TAboutForm.imgLazarusClick(Sender: TObject);
begin
AppBusy;
OpenURL('http://www.lazarus.freepascal.org');
OpenURL('https://www.lazarus-ide.org');
Copy link
Member

Choose a reason for hiding this comment

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

This one seems to be good 👍

@PeterDaveHello PeterDaveHello changed the title Fix urls to avoid redirects Update urls in "about" to avoid redirects Jul 24, 2017
@PeterDaveHello PeterDaveHello merged commit 96c6aea into transmission-remote-gui:master Jul 24, 2017
@duramato duramato deleted the patch-1 branch July 24, 2017 10:43
@PeterDaveHello
Copy link
Member

Landed on v5.8.3 here: https://github.com/leonsoft-kras/transmisson-remote-gui/releases/tag/v5.8.3, thanks for your contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants