Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Website Docs: Remove unused pages #2974

Merged
merged 3 commits into from
Apr 8, 2022
Merged

Conversation

BigRoy
Copy link
Collaborator

@BigRoy BigRoy commented Mar 30, 2022

Brief description

Removes unused website/docs pages.

Description

I solely did a quick search manually so this might be false positives for "unused". Please double check.

Additional info

Artist Hosts Nuke - two nearly identical files, one old, one newer. Remove one?

There are also two files artist_hosts_nuke.md and artist_hosts_nuke_tut.md which appear very similar. However, in some odd in-page links it links to artist_hosts_nuke.md.

Should we remove artist_host_nuke.md (older) and replace it with artist_hosts_nuke_tut.md (newer) and maybe even refactor that name back to artist_host_nuke.md?

Testing notes:

  1. please confirm these pages were indeed unused and redundant

@BigRoy
Copy link
Collaborator Author

BigRoy commented Mar 30, 2022

This admin_webserver_for_webpublisher.md page also appears unused in the actual website docs. Should this be exposed? Or is that just still temporarily hidden?

@m-u-r-p-h-y
Copy link
Member

This admin_webserver_for_webpublisher.md page also appears unused in the actual website docs. Should this be exposed? Or is that just still temporarily hidden?

should probably be exposed and added to sidebars.js
what was your plan @kalisp ?

@mkolar
Copy link
Member

mkolar commented Apr 5, 2022

Webpublisher should actually be removed completely at this point from the repo as well and turned into a studio specific addon until it's in a better state. Right now it's quite heavily customised to a single site and it's not the best idea to have it part of the main project to be honest.

@kalisp
Copy link
Member

kalisp commented Apr 7, 2022

Lets keep admin_webserver_for_webpublisher.md there for now, it will be consumed (translated into readme, I guess) in OP-2889.

@jrsndl
Copy link
Collaborator

jrsndl commented Apr 7, 2022

File artist_hosts_nuke_tut.md includes all content from artist_host_nuke.md, so artist_host_nuke.md can be removed.
The in-page link to artist_hosts_nuke.md is a mistake.

@BigRoy
Copy link
Collaborator Author

BigRoy commented Apr 7, 2022

Feel free to push changes into this PR if you have write access to OpenPype you should be able to.

remove  artist_host_nuke.md, fix in-page link to artist_hosts_nuke.md
@kalisp kalisp merged commit 5de8185 into ynput:develop Apr 8, 2022
@mkolar mkolar added this to the next milestone Apr 8, 2022
@BigRoy BigRoy deleted the documentation_cleanup branch March 20, 2024 15:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants