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

change some links to https from http #2647

Closed
wants to merge 1 commit into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion deploy/files/prod-public-assets/maintenance.html
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ <h1>We&rsquo;ll be back soon!</h1>
<br>
<p>Follow us on <a href="https://twitter.com/trustroots">Twitter</a> and <a href="https://www.facebook.com/trustroots.org">Facebook</a></p>
<br>
<p>Check our <a href="http://status.trustroots.org/">status page</a></p>
<p>Check our <a href="https://status.trustroots.org/">status page</a></p>
Copy link
Contributor

@mrkvon mrkvon Sep 30, 2023

Choose a reason for hiding this comment

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

👌🏾 This one doesn't seem to function properly. In any case https is better than http. Review and fixing of links can be done in separate PR.

</div>
</article>

Expand Down
2 changes: 1 addition & 1 deletion modules/core/client/controllers/app.client.controller.js
Original file line number Diff line number Diff line change
Expand Up @@ -215,7 +215,7 @@ function AppController(
' <p>We expect to be back in a couple minutes. Thanks for your patience.</p>' +
' <p>In the meantime, check our ' +
' <a href="https://twitter.com/trustroots" target="_blank">Twitter account</a> or ' +
' <a href="http://status.trustroots.org/" target="_blank">status page</a> for news.</p>' +
' <a href="https://status.trustroots.org/" target="_blank">status page</a> for news.</p>' +
'</div>' +
'<div class="modal-footer">' +
' <button class="btn btn-primary" type="button" ng-click="$dismiss()">OK</button>' +
Expand Down
22 changes: 11 additions & 11 deletions modules/core/client/fonts/fontello/LICENSE.txt
Original file line number Diff line number Diff line change
Expand Up @@ -7,51 +7,51 @@ Font license info

Author: Dave Gandy
License: SIL ()
Homepage: http://fortawesome.github.com/Font-Awesome/
Homepage: https://fortawesome.github.com/Font-Awesome/
Copy link
Contributor

@mrkvon mrkvon Sep 30, 2023

Choose a reason for hiding this comment

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

👌🏾 This one doesn't exist anymore, but i'm not sure what would be a proper link. 🤷🏾‍♀️



## Modern Pictograms

Copyright (c) 2012 by John Caserta. All rights reserved.

Author: John Caserta
License: SIL (http://scripts.sil.org/OFL)
Homepage: http://thedesignoffice.org/project/modern-pictograms/
License: SIL (https://scripts.sil.org/OFL)
Homepage: https://thedesignoffice.org/project/modern-pictograms/


## Entypo

Copyright (C) 2012 by Daniel Bruce

Author: Daniel Bruce
License: SIL (http://scripts.sil.org/OFL)
Homepage: http://www.entypo.com
License: SIL (https://scripts.sil.org/OFL)
Homepage: https://www.entypo.com


## Elusive

Copyright (C) 2013 by Aristeides Stathopoulos

Author: Aristeides Stathopoulos
License: SIL (http://scripts.sil.org/OFL)
Homepage: http://aristeides.com/
License: SIL (https://scripts.sil.org/OFL)
Homepage: https://aristeides.com/
Copy link
Contributor

@mrkvon mrkvon Sep 30, 2023

Choose a reason for hiding this comment

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

⬇️ https page has error, and http redirects to https://aristath.github.io/, so maybe that's the proper link?

Anyways, it would make sense to either keep the http, or use the new https link



## MFG Labs

Copyright (C) 2012 by Daniel Bruce

Author: MFG Labs
License: SIL (http://scripts.sil.org/OFL)
Homepage: http://www.mfglabs.com/
License: SIL (https://scripts.sil.org/OFL)
Homepage: https://www.mfglabs.com/


## Zocial

Copyright (C) 2012 by Sam Collins

Author: Sam Collins
License: MIT (http://opensource.org/licenses/mit-license.php)
Homepage: http://zocial.smcllns.com/
License: MIT (https://opensource.org/licenses/mit-license.php)
Homepage: https://zocial.smcllns.com/


4 changes: 2 additions & 2 deletions modules/messages/tests/server/message.server.routes.tests.js
Original file line number Diff line number Diff line change
Expand Up @@ -344,7 +344,7 @@ describe('Message CRUD tests', function () {
// Create html in message
const htmlMessage = message;
htmlMessage.content =
'<strong>strong</strong><br><img src="http://www.trustroots.org/">' +
'<strong>strong</strong><br><img src="https://www.trustroots.org/">' +
'<foo>blockquote</foo><p>' +
'<script></script>' +
'<a href="https://www.trustroots.org/">link</a>' +
Expand Down Expand Up @@ -380,7 +380,7 @@ describe('Message CRUD tests', function () {
'<br />blockquote' +
'<p>' +
'<a href="https://www.trustroots.org/">link</a>' +
'<a href="http://www.trustroots.org">www.trustroots.org</a>' +
'<a href="https://www.trustroots.org">www.trustroots.org</a>' +
Copy link
Contributor

Choose a reason for hiding this comment

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

This test fails: the original link www.trustroots.org actually formats to <a href="http://www.trustroots.org">www.trustroots.org</a>

(to http)

So this change should be reverted, in order for tests to pass again.

' </p>';

thread[0].content.should.equal(output);
Expand Down
2 changes: 1 addition & 1 deletion modules/pages/client/components/FaqFoundation.component.js
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,7 @@ export default function FaqFoundation() {
<a href="/contribute">Donations</a> and grants are the most likely
ways. There are many inspiring organisations out there serving as
great examples:{' '}
<a href="http://www.abgefahren-ev.de/">Abgefahren e.V.</a>{' '}
<a href="https://www.abgefahren-ev.de/">Abgefahren e.V.</a>{' '}
Copy link
Contributor

@mrkvon mrkvon Sep 30, 2023

Choose a reason for hiding this comment

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

⬇️ This one doesn't seem to have https version. Only http.

(Hitchwiki), <a href="https://www.bevolunteer.org/">BeVolunteer</a>,{' '}
<a href="https://servas.org/">Servas</a>,{' '}
<a href="https://wwoof.net/">WWOOF</a> and many more. Again, if you
Expand Down
2 changes: 1 addition & 1 deletion modules/pages/client/components/FaqGeneral.component.js
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ export default function FaqGeneral() {
<Trans t={t} ns="pages">
If you would like to help out developing our mobile apps, read more
about{' '}
<a href="http://team.trustroots.org/" rel="noopener">
<a href="https://team.trustroots.org/" rel="noopener">
developing Trustroots
</a>{' '}
. <br />
Expand Down
4 changes: 2 additions & 2 deletions modules/pages/client/components/FaqTechnology.component.js
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,9 @@
<h3>{t("I'm a developer and I want to help!")}</h3>
<Trans t={t} ns="pages">
Great! Read about{' '}
<a href="http://team.trustroots.org/">development</a> and get in touch
<a href="https://team.trustroots.org/">development</a> and get in touch

Check failure on line 27 in modules/pages/client/components/FaqTechnology.component.js

View workflow job for this annotation

GitHub Actions / ESLint Report Analysis

modules/pages/client/components/FaqTechnology.component.js#L27

[prettier/prettier] Delete `·touch`
with us! We are looking for JavaScript developers (back- and frontend)
and sysops.

Check failure on line 29 in modules/pages/client/components/FaqTechnology.component.js

View workflow job for this annotation

GitHub Actions / ESLint Report Analysis

modules/pages/client/components/FaqTechnology.component.js#L28-L29

[prettier/prettier] Replace `·with·us!·We·are·looking·for·JavaScript·developers·(back-·and·frontend)⏎·········` with `·touch·with·us!·We·are·looking·for·JavaScript·developers·(back-·and⏎··········frontend)`
</Trans>
</div>

Expand All @@ -42,7 +42,7 @@
limited software you can imagine that still works for hospitality. We
simply haven&apos;t had time to build most of the features yet. Please
be patient and be mindful about{' '}
<a href="http://www.productstrategymeanssayingno.com/">
<a href="https://www.productstrategymeanssayingno.com/">
Copy link
Contributor

@mrkvon mrkvon Sep 30, 2023

Choose a reason for hiding this comment

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

⬇️ The https version hangs for me at the moment. Only the http link redirects to a meaningful website atm.

feature creep
</a>
.
Expand Down
2 changes: 1 addition & 1 deletion modules/pages/client/components/Foundation.component.js
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ export default function Foundation({ user }) {
<Trans t={t} ns="pages">
Working on big freegan/travel projects such as{' '}
<a href="https://hitchwiki.org/">Hitchwiki</a>,{' '}
<a href="http://hitchgathering.org/">
<a href="https://hitchgathering.org/">
Copy link
Contributor

@mrkvon mrkvon Sep 30, 2023

Choose a reason for hiding this comment

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

⬇️ The https version hangs for me. Only the http version seems to redirect to a meaningful page

Hitchgathering festival
</a>
, <a href="https://trashwiki.org/">Trashwiki</a> and{' '}
Expand Down
2 changes: 1 addition & 1 deletion modules/pages/client/components/Home.component.js
Original file line number Diff line number Diff line change
Expand Up @@ -396,7 +396,7 @@ export default function Home({ user, isNativeMobileApp, photoCredits }) {
<p>
<i className="icon-right"></i>
<a
href="http://team.trustroots.org/"
href="https://team.trustroots.org/"
className="home-footer-more font-brand-semibold"
>
{t('For developers')}
Expand Down
2 changes: 1 addition & 1 deletion public/maintenance/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ <h1>We&rsquo;ll be back soon!</h1>
otherwise we&rsquo;ll be back online shortly!</p>
<p>&mdash; The Trustroots Team</p>
<br>
<p>Check our <a href="http://status.trustroots.org/">status page</a> for updates
<p>Check our <a href="https://status.trustroots.org/">status page</a> for updates
or follow us on <a href="https://twitter.com/trustroots">Twitter</a>
and <a href="https://www.facebook.com/trustroots.org">Facebook</a>.</p>
<!--<p><em><small class="gray">Updated 02:52 CEST</small></em></p>-->
Expand Down
Loading