Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Replacement Text: Barbarian and Bard #285

Closed
Gwaithador opened this issue Apr 3, 2016 · 5 comments
Closed

Replacement Text: Barbarian and Bard #285

Gwaithador opened this issue Apr 3, 2016 · 5 comments
Assignees
@Sklore
Labels
Enhancement

Comments

@Gwaithador
Copy link

SRD Community File Replacement Text 3-26-2016.docx
@Gwaithador
Copy link
Author

I developed replacement text for the barbarian and bard. If this meets the 20% standard, I'll do some more. Please let me know if this acceptable.

@dschwarm
Copy link

dschwarm commented Apr 3, 2016

Great work! This looks fine to me.

I am not sure how we "validate" the 20% rule...Your barbarian and bard things look fine to me, but I assume we should run it through some kind of text comparison tool and save the results in case of audit? I am sure there is precedence on this, I just do not know what it is.

@Sklore
Copy link
Owner

Sklore commented Apr 3, 2016

Absolutely no idea if there is any formal way to do it. So far we have just been making the necessary changes and releasing the new text. As long as we verify that the text is satisfactorily different we should be ok to proceed to update.

@Sklore Sklore self-assigned this Apr 4, 2016
Sklore pushed a commit that referenced this issue Apr 4, 2016
Update COM_5ePack_PHB - Classes.user
5ff1468 
#285 - Updated placeholder text with paraphrased text for Bard and Barbarian class options.
@Sklore Sklore mentioned this issue Apr 4, 2016
Merged
Sklore referenced this issue Apr 4, 2016
Merge pull request #286 from Sklore/Daplunk---#285
63ea43d 
Update COM_5ePack_PHB - Classes.user
@Sklore
Copy link
Owner

Sklore commented Apr 4, 2016

Love your work :) This will be released in the next update.

@Sklore Sklore closed this as completed Apr 4, 2016
Sklore referenced this issue Apr 4, 2016
Merge pull request #287 from Sklore/Daplunk---#285
e0a02dc 
Update COM_5ePack_Private_MH5e - All.user
Sklore referenced this issue Apr 4, 2016
Merge pull request #288 from Sklore/Daplunk---#285
8a11249 
Update COM_5ePack_Private_MH5e - All.user
@Gwaithador
Copy link
Author

Thanks gents. I'll start on the cleric, and proceed from there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Sklore Sklore

Labels
Enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Sklore @Gwaithador @dschwarm