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.

Sign up for GitHub

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

Jump to bottom

fix: IFPS #1032

Merged
merged 4 commits into from
Feb 22, 2022
Merged

fix: IFPS #1032

merged 4 commits into from
Feb 22, 2022

Conversation

lidel
Copy link
Member

@lidel lidel commented Feb 17, 2022

🥀 IFPS → 🌹 IPFS

@lidel lidel requested a review from johnnymatthews February 17, 2022 23:43
@filecorgi
Copy link
Contributor

  • Image optimization came back clean!
  • Vuepress build was successful!

docs/community/contribute/grammar-formatting-and-style.md:

  • The following link returned HTTP status code 302 (Found):
    https://proto.school/events
  • The following link returned HTTP status code 301 (Moved Permanently):
    https://www.meetup.com/members/249142444/
  • The following link returned HTTP status code 405 (Method Not Allowed):
    https://www.grammarly.com/
  • The following link returned HTTP status code 302 (Found):
    https://github.com/DavidAnson/markdownlint/blob/master/doc/Rules.md
  • Only proper nouns start with an uppercase character (there are exceptions for headlines). 
    ...n words like center. # The Oxford comma Follow each list of three or more it...
                                           ^^^^^^
  • Although a hyphen is possible, it is not necessary in a compound modifier in which the first word is an adverb that ends in ‘ly’. 
    ...ion: This generally isn't necessary for commonly-encountered acronyms like I...
                                           ^^^^^^^^^^^^^^^^^^^^
  • When ‘Project-specific’ is used as a modifier, it is usually spelled with a hyphen. 
    ...chine (VM), Decentralized Web (DWeb). # Project specific titles When referri...
                                           ^^^^^^^^^^^^^^^^
  • Did you mean the formatting language “Markdown” (= proper noun)? 
    ...the GitHub Flavored Markdown syntax for markdown. This way, all articles dis...
                                           ^^^^^^^^
  • An apostrophe may be missing. 
    ...ders the option to view articles on the docs website  (opens new window) or ...
                                           ^^^^
  • Two consecutive dots 
    ...hem using full relative paths (e.g. use ../ for climbing a directory) and sp...
                                           ^^
  • As a shorter alternative for ‘able to’, consider using “can”. 
    ...ent directly in-repo on GitHub's web UI are able to follow relative links co...
                                           ^^^^^^^^^^^
  • Unpaired symbol: ‘"’ seems to be missing 
    ...italics and surrounded by double quotes ": # Code blocks Tag code blocks wit...
                                           ^
  • Only proper nouns start with an uppercase character (there are exceptions for headlines). 
    ...are presenting: # Command-line examples Write command-line inputs without an...
                                           ^^^^^
  • Only proper nouns start with an uppercase character (there are exceptions for headlines). 
    ...raneous information: # Inline code tags Surround directories, file names, an...
                                           ^^^^^^^^
  • Only proper nouns start with an uppercase character (there are exceptions for headlines). 
    ...rom Father Christmas. # Unordered lists Use the dash character - for un-numb...
                                           ^^^
  • This phrase is redundant. Consider using “outside”. 
    ...s in code tags: The plus symbol + stays outside of the code tags. # Images T...
                                           ^^^^^^^^^^
  • An apostrophe may be missing. 
    ...ticle, so there is no folder within the images directory for that article. #...
                                           ^^^^^^

docs/how-to/companion-window-ipfs.md:

  • The following link returned HTTP status code 301 (Moved Permanently):
    Meta: window.ipfs v2 ipfs-companion#589
  • The following link returned HTTP status code 301 (Moved Permanently):
    Expose IPFS API as window.ipfs ipfs-companion#330
  • The following link returned HTTP status code 301 (Moved Permanently):
    https://github.com/ipfs/js-ipfs-http-client
  • The following link returned HTTP status code 301 (Moved Permanently):
    https://github.com/tableflip/ipfs-postmsg-proxy
  • The following link returned HTTP status code 301 (Moved Permanently):
    https://github.com/ipfs-shipyard/ipfs-companion/blob/master/examples/window.ipfs-fallback.html
  • The following link returned HTTP status code 504 (Gateway Timeout):
    https://ipfs.io/ipfs/QmTegrragyzfFq6DSuUaPYoKzm4eRBj2tgQaDHC72dLLaV/
  • The following link returned HTTP status code 301 (Moved Permanently):
    https://github.com/ipfs/interface-ipfs-core/blob/master/SPEC/FILES.md
  • Only proper nouns start with an uppercase character (there are exceptions for headlines). 
    ...ew window), as of October 19, 2020, and Remove legacy window.ipfs  (opens ne...
                                           ^^^^^^
  • Possible missing article found. 
    ...e changes of this interface. It reduces amount of code needed to implement a...
                                           ^^^^^^
  • An apostrophe may be missing. Consider inserting a curly or a straight apostrophe. (Some terms such as “farmers’ market” have a possessive apostrophe after the first noun.) 
    ...denied for a specific command, then the permissions list is present and incl...
                                           ^^^^^^^^^^^
  • An apostrophe may be missing. Consider inserting a curly or a straight apostrophe. (Some terms such as “farmers’ market” have a possessive apostrophe after the first noun.) 
    ...roxy was disabled by the user, then the permissions list is missing entirely...
                                           ^^^^^^^^^^^
  • The verb ‘depend’ requires the preposition ‘on’ (or ‘upon’). 
    ...les you to obtain an IPFS API instance. Depending how IPFS Companion is conf...
                                           ^^^^^^^^^
  • Possible missing article found. 
    ...ntially others in the future. Note that object returned by window.ipfs.enabl...
                                           ^^^^^^
  • For conciseness, consider replacing this expression with an adverb. 
    .... # What about IPFS node configuration? Right now, access to the config comm...
                                           ^^^^^^^^^
  • As a shorter alternative for ‘able to’, consider using “can”. 
    ...ontrol (ACL)? Yes. IPFS Companion users are able to selectively control acce...
                                           ^^^^^^^^^^^
  • You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing. 
    ... confirm every API call? Command access needs to be confirmed only once per ...
                                           ^^^^^^^^
  • Possible missing comma found. 
    ...ble to call previously whitelisted IPFS commands and users will not be promp...
                                           ^^^^^^^^
  • If a new sentence starts here, add a space and start with an uppercase letter. 
    ...f permissions. For example: Allow files.add to https://domain.com/ ...will a...
                                           ^^^
  • If a new sentence starts here, add a space and start with an uppercase letter. 
    ...https://domain.com/ ...will allow files.add to: https://domain.com/file http...
                                           ^^^
  • If a new sentence starts here, add a space and start with an uppercase letter. 
    ...in.com/sub/paths/files etc. Allow files.add to https://domain.com/feature .....
                                           ^^^
  • If a new sentence starts here, add a space and start with an uppercase letter. 
    .../domain.com/feature ...will allow files.add to: https://domain.com/feature/f...
                                           ^^^
  • If a new sentence starts here, add a space and start with an uppercase letter. 
    ....will cause additional prompt for files.add to: https://domain.com/ https://...
                                           ^^^
  • If a new sentence starts here, add a space and start with an uppercase letter. 
    ...lication is running. For example: files.write to /myfile.txt on https://doma...
                                           ^^^^^
  • If a new sentence starts here, add a space and start with an uppercase letter. 
    ...dapps/https/domain.com/myfile.txt files.write to /path/to/myfile.txt on http...
                                           ^^^^^
  • If a new sentence starts here, add a space and start with an uppercase letter. 
    ...in.com/feature/path/to/myfile.txt files.read from /feature/path/to/myfile.tx...
                                           ^^^^
  • If a new sentence starts here, add a space and start with an uppercase letter. 
    ...in.com/feature/path/to/myfile.txt files.stat to / on https://domain.com/feat...
                                           ^^^^
  • If a new sentence starts here, add a space and start with an uppercase letter. 
    ...s /dapps/https/domain.com/feature files.read from /../myfile.txt on https://...
                                           ^^^^
  • Two consecutive dots 
    ...tps/domain.com/feature files.read from /../myfile.txt on https://domain.com/...
                                           ^^

docs/how-to/websites-on-ipfs/static-site-generators.md:

  • The following link returned HTTP status code 302 (Found):
    https://blog.fleek.co/posts/wordpress+fleek
  • Possible missing comma found. 
    ...tall and set up your project. In config.toml add relativeURLS and set it to ...
                                           ^^^^
  • Only proper nouns start with an uppercase character (there are exceptions for headlines). 
    ... and set it to true. Build static pages Output will be in ./public/ director...
                                           ^^^^^^
  • Possible missing preposition found. 
    ... using all-relative  (opens new window) Upload the dist folder to IPFS. # Mi...
                                           ^^^^^^
  • Possible missing preposition found. 
    ... dist folder to IPFS. # Middleman Refer the Middleman's Installation  (opens...
                                           ^^^
  • Did you mean the formatting language “Markdown” (= proper noun)? 
    ...s generated by the link_to helper or by markdown will become relative. Build...
                                           ^^^^^^^^
  • Possible missing preposition found. 
    ... using all-relative  (opens new window) Upload the _site folder to IPFS. # W...
                                           ^^^^^^

@johnnymatthews johnnymatthews merged commit b166a3c into main Feb 22, 2022
@johnnymatthews johnnymatthews deleted the fix/ifps branch February 22, 2022 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@johnnymatthews johnnymatthews johnnymatthews approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lidel @filecorgi @johnnymatthews