Skip to content

Update readme changes#48

Merged
rpavlik merged 2 commits intomasterfrom
updateReadmeChanges
Jun 4, 2015
Merged

Update readme changes#48
rpavlik merged 2 commits intomasterfrom
updateReadmeChanges

Conversation

@DuFF14
Copy link
Copy Markdown
Member

@DuFF14 DuFF14 commented May 31, 2015

Updated Changes.md to reflect recent changes. Updated Readme.md to provide link to documentation, as requested in Unity Asset Store submission feedback.

@DuFF14
Copy link
Copy Markdown
Member Author

DuFF14 commented Jun 2, 2015

Unity Asset Store asks for "a full documentation file or a readme file that tells users where to find your online documentation." Do you think a link to the repo Readme will suffice? Is there a more appropriate link?
Once this is merged, I will resubmit the package to the Asset Store.

@rpavlik
Copy link
Copy Markdown
Member

rpavlik commented Jun 4, 2015

Well, I think that's the best we have so far, though it's not ideal.

@rpavlik
Copy link
Copy Markdown
Member

rpavlik commented Jun 4, 2015

How hard is updating once we submit? Curious if you should be waiting until after I get some changes in.

rpavlik added a commit that referenced this pull request Jun 4, 2015
@rpavlik rpavlik merged commit a54da1b into master Jun 4, 2015
@rpavlik rpavlik deleted the updateReadmeChanges branch June 4, 2015 15:42
@DuFF14
Copy link
Copy Markdown
Member Author

DuFF14 commented Jun 4, 2015

It's not hard to update, just uploading the .unitypackage. It might take a week for an update to be approved/processed.

@rpavlik
Copy link
Copy Markdown
Member

rpavlik commented Jun 4, 2015

ah, so that's a pain. What about from the user end - how easy to update to a new version to an asset you got from the store?

@DuFF14
Copy link
Copy Markdown
Member Author

DuFF14 commented Jun 4, 2015

Easy. Just hitting an update button from the download manager, and it shows you which files have changed. You can select which files you want to import.

@rpavlik
Copy link
Copy Markdown
Member

rpavlik commented Jun 4, 2015

OK, so at least there's that, that's OK. Then I think it's probably OK to send in the current version.

@DuFF14
Copy link
Copy Markdown
Member Author

DuFF14 commented Jun 4, 2015

Hmm, that Readme doesn't get included in the unitypackage, so I will add it manually for this upload.

@rpavlik
Copy link
Copy Markdown
Member

rpavlik commented Jun 4, 2015

oh, it's supposed to? Then that's easy to do - just edit the build script, it's the only editor script we have.

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.

2 participants