Skip to content

overhauled the docs to remove dead links - #13

Merged
qreeves merged 8 commits into
redeclipse:masterfrom
nochinator:master
Feb 10, 2023
Merged

qreeves merged 8 commits into
redeclipse:masterfrom
nochinator:master

Conversation

@nochinator

Copy link
Copy Markdown
Contributor

Changes proposed in this request:

  • fixed most (may have missed a few as there were a lot) of broken links
  • updated pages for the weapons
  • fixed some grammar or spelling errors
  • wrote pages for every game mode
  • connected most (may have missed a few) pages together in a more fluid way
  • incorporated many unused, or hard-to-find pages that had lots of useful info into the flow of the website
  • organized file system into folders, gameplay, editing, server management, and variables
  • added variables from https://github.com/redeclipse/base/blob/master/config/usage.cfg into this, although it is not a complete list it's much better than nothing. This will be sorted into better categories and polished up in the future.

Comment thread variables/Variables-home.md Resolved
@nochinator nochinator changed the title Massively overhauled the docs to remove dead links overhauled the docs to remove dead links Jan 31, 2023

@qreeves qreeves left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Other than the review comments attached, the only other thing I'd suggest is that if you're going to move the editing documents into their own folder, they don't really need the Editing- prefix on the filenames themselves anymore as it's a duplication.

Comment thread .vscode/settings.json Resolved
Comment thread FAQ.md Resolved
Comment thread gameplay/Capture-the-Flag.md Resolved
Comment thread gameplay/Gameplay-Guide.md Resolved
Comment thread gameplay/Gameplay-Guide.md Resolved
Comment thread gameplay/Race.md Resolved
Comment thread gameplay/Status-Effects.md Resolved
Comment thread gameplay/Status-Effects.md Resolved
Comment thread gameplay/Weapons-Guide.md Resolved
Comment thread sever-how-to/Server-Setup.md Resolved
@qreeves

qreeves commented Feb 2, 2023

Copy link
Copy Markdown
Member

You'll need to revert .gitignore before I can merge as I've already changed it upstream to include more matches.

image

Otherwise, looks good to me, ready for merge once resolved.

@nochinator

Copy link
Copy Markdown
Contributor Author

I've fixed the problem and this should be ready to merge. I can't figure out how to get the "1 change requested" to go away, but I fixed what it says the change is.

@qreeves qreeves left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yeah it's still broken 😛

Comment thread .gitignore
*.orig
*.rej
*.bak
*.kate-swp

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is reverting changes I made upstream to ignore the .vscode directory and other unwanted files.

@nochinator nochinator Feb 9, 2023 •

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

oh, well ok then, re-added

Comment thread .vscode/settings.json Outdated
"unconnectable",
"Unported"
]
} No newline at end of file

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This file is still present in the latest revision.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I have removed the file

@qreeves

qreeves commented Feb 10, 2023

Copy link
Copy Markdown
Member

All good now, thanks for your contribution and cooperation.

@qreeves
qreeves merged commit 70bbd87 into redeclipse:master Feb 10, 2023
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.

3 participants