Skip to content

Add link to a separate assets repository #470

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

Closed
wants to merge 2 commits into from
Closed

Conversation

Tzky
Copy link
Contributor

@Tzky Tzky commented Mar 25, 2016

Once the SpongeAssets repository becomes available to the public, this PR will adjust the Art Assets page to redirect users.

Solves #457

@Spongy
Copy link

Spongy commented Mar 25, 2016

A preview for this pull request is available at https://cdn.rawgit.com/Spongy/SpongeDocs-PRs/f62dac32/index.html.

Here are some links to the pages that were modified:

Since the preview frequently changes, please link to this comment, not to the direct url to the preview.

@Tzky
Copy link
Contributor Author

Tzky commented Mar 25, 2016

I'm not sure if we should keep the usage rules on the Docs or not as they'd better fit on the new assets repo.

Thoughts?

@Tzky Tzky added the needs review The submission is ready and needs to be reviewed label Mar 25, 2016
@Inscrutable
Copy link
Member

The usage rules should get reviewed when the new repo is ready to go public, but I don't see the harm in having it reproduced. That way they will get translated, and if this is the staging point directing people there, then keeping them is still appropriate.

@Tzky
Copy link
Contributor Author

Tzky commented Mar 26, 2016

Agreed. I'll keep them, but they must be updated before we pull this PR in.

We should also pull this PR into the static branches, not much extra effort, but keeps things clean.

@Inscrutable
Copy link
Member

We need to determine what license we're using. I think it'll be Creative Commons (CC0) or possibly CC-ND (non-derivative) or even CC-NC (non-commercial)�. I'll let the overlords decide that one.

@Tzky Tzky added on hold Progress is blocked for some reason and removed needs review The submission is ready and needs to be reviewed labels Mar 28, 2016
@Tzky Tzky self-assigned this May 9, 2016
@meyerzinn
Copy link

Any update on this?

@Tzky
Copy link
Contributor Author

Tzky commented Jul 12, 2016

We're still working out the details of the SpongeAssets repo. Afaik it's a private repo currently.

@stephan-gh stephan-gh changed the base branch from master to stable February 4, 2017 09:57
@Tzky
Copy link
Contributor Author

Tzky commented Feb 4, 2017

Bump. With the relicensing of the docs currently in progress, we should get this resolved too. Are there any plans to get a repo up with all assets?

@Inscrutable Inscrutable added the FLARD This is gathering dust label Jul 29, 2017
@Inscrutable
Copy link
Member

This ancient PR needs a clean up if it's ever going to happen.
The assets repo in the link does not exist, and in any case I think we should largely replace the contents of the page with excerpts from and a link to the Sponge Branding Manual. Anyone want to help here? Someone's arm needs to get twisted to get the manual formally approved, and I note that it's content is not quite in keeping with what the docs say at present. We also need to find or create new links to master graphics assets, if we're ever going to officially offer them.
Pinging the absent @Tzky

@Inscrutable Inscrutable added help wanted We would appreciate contributions input wanted We would like to hear your opinion labels Oct 10, 2017
@kashike
Copy link
Contributor

kashike commented Jun 29, 2018

x

@Inscrutable
Copy link
Member

This is currently a dead duck until @Mumfrey gives it some love.
A new PR can get made when that eventually happens. Or ragnarok.

@Inscrutable Inscrutable closed this Jul 1, 2018
ST-DDT added a commit that referenced this pull request Nov 21, 2018
ST-DDT added a commit that referenced this pull request Nov 21, 2018
@ST-DDT
Copy link
Member

ST-DDT commented Nov 21, 2018

I added most of the stuff from everything till b898b28 in #776

@ST-DDT ST-DDT deleted the feature/art-assets branch November 21, 2018 19:48
Inscrutable pushed a commit that referenced this pull request Jan 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FLARD This is gathering dust help wanted We would appreciate contributions input wanted We would like to hear your opinion on hold Progress is blocked for some reason
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants