chore: deprecate edx-sphinx-theme #763
Conversation
docs/conf.py
Outdated
| href="https://openedx.org" | ||
| property="cc:attributionName" | ||
| rel="cc:attributionURL" | ||
| >Axim Collaborative</a> |
There was a problem hiding this comment.
Is this appropriate for edx-arch-experiments?
There was a problem hiding this comment.
Hey @timmc-edx, you are absolutely right about the Author, license and copyright information. Can I please have the exact data that we need to now add in license, author and copyright against this repo?
There was a problem hiding this comment.
I heard back from Legal -- we should use edX LLC for copyright. As far as licensing, I believe it's correct to leave the docs at cc-by-sa-4, because docs.edx.org uses that in its footer. I'll check in with legal on that asynchronously, though. (It's probably not correct to have the repo licensed as AGPL or Apache but the published docs as CC, but that's probably going to be a longer conversation...)
huniafatima-arbi
force-pushed
the
huniafatima/deprecate-edx-sphinx-theme
branch
from
c8215e6
to
e0689b4
Compare
|
Hey @timmc-edx, I have updated this PR. Can you please give it a look and see if it's good to be merged? |
|
I'm still seeing Axim listed rather than edX in at least one place. Good question on the logo, but I don't actually know... |
huniafatima-arbi
force-pushed
the
huniafatima/deprecate-edx-sphinx-theme
branch
from
0a87e51
to
ed51eb0
Compare
Hey @timmc-edx, confirmed with @dianakhuang we shall be handling logo in a separate issue. |
This PR deprecates the
edx-sphimx-themeand replaces it withsphinx-book-themeIssue Link: #339