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

Fix URLs to websites and resources #18844

Merged
merged 4 commits into from
Jul 30, 2020

Conversation

Diegovd
Copy link
Contributor

@Diegovd Diegovd commented Jul 29, 2020

Summary:

1.- I got a reply from ERE with the new document filename and thus could find the new correct URL for that resource.

2.- Some corrections from #18819 seem not to have been integrated into the base bugfix-2.0.x so that is why I am resubmitting them. Hope that is OK.

I have corrected instances where the URL of a website or a document was written with http and replaced with httpS when the secure URL was available. I also corrected moved websites and documents.
This concerns mainly URLs in Annotations, some in Parameters shown on screen. No changes in CODE.

Note: Changed the syntax of some links from "( )" to "< >" to avoid, while editing in Notepad++, that a double-click on an URL opens the URL with ")" at the end (=> Error 404)
Example:
BEFORE: * direct port writing by Mikal Hart (http://www.arduiniana.org)
AFTER: * direct port writing by Mikal Hart "<"http://www.arduiniana.org">"

Testing: tested all modifications during editing. Visually checked all changes before commit.

Thank you, regards, Diego von Deschwanden.

Benefits

This provides accurate and secure URLs in the Annotations.
This updates deprecated URLs and Not Found ressources.
This reduces latency when navigating to the desired website or document.

Related Issues
-> No related Issues

Diegovd and others added 4 commits July 28, 2020 06:45
Summary: This is the second pass:

I have corrected many instances where the URL of a website or a document was written with http://,
and replaced with httpS when the secure URL was available. I also corrected moved websites and documents.

This concerns mainly URLs in Annotations, some in Parameters shown on screen. No changes in CODE.

Many files changes relate to the GNU Licenses URL.

Note: I left some "http" still stand because there is no httpS, or I could not find the appropriate replacement URL.

Thanks

Benefits

This provides accurate and secure URLs in the Annotations.
This reduces latency when navigating to the desired website or document.
Related Issues

No related Issues
**Summary:**
I got a reply from ERE with the new document filename and thus could find the new correct URL for that ressource.

**Benefits**

This updates deprecated URLs and Not Found ressources.

**Related Issues**
-> No related Issues
@thinkyhead
Copy link
Member

Note that it's highly likely that even the default conf files provided by the keepers of avrdude will tend to have out of date links, so how much we want to be the keepers of the links for these, I dunno….

@thinkyhead thinkyhead merged commit e549d47 into MarlinFirmware:bugfix-2.0.x Jul 30, 2020
@Diegovd
Copy link
Contributor Author

Diegovd commented Jul 30, 2020

Hi, sure, but since nothing lasts forever, and nothing can be perfect in this world... At leat I/we tried.

I am happy that I am learning new things, GitHub and such. If I can help at my level with small things that is OK.
If they stay or not... Well, nobody is staying on this earth long enough anyway ;)

Two months ago I had never used a 3D printer so...
Cheers mate ;)

domrim pushed a commit to domrim/Marlin that referenced this pull request Aug 2, 2020
Speaka pushed a commit to Speaka/Marlin that referenced this pull request Aug 13, 2020
albertogg pushed a commit to albertogg/Marlin that referenced this pull request Aug 31, 2020
vgadreau pushed a commit to vgadreau/Marlin that referenced this pull request Dec 9, 2020
ahmetcemturan pushed a commit to ahmetcemturan/Marlin that referenced this pull request Jan 21, 2021
ahmetcemturan added a commit to ahmetcemturan/Marlin that referenced this pull request Jan 23, 2021
kageurufu pushed a commit to CR30-Users/Marlin-CR30 that referenced this pull request Apr 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants