Skip to content

Conversation

@eheimburg
Copy link
Contributor

The text-fields for mage abilities just got wiped out somewhere along the way. I've filled in basic descriptor text. Also fixed rogue base attack, which was similarly missing text.

The text-fields for these abilities just got wiped somewhere along the way. I've filled in basic descriptor text. Also fixed rogue base attack, which was similarly missing text.
@eheimburg eheimburg added the 2-Easy This PR is trivial and can be reviewed quickly label Mar 22, 2021
@SamuelBellomo SamuelBellomo added the 1-Needs Review PR needs attention from the assignee and reviewers label Mar 22, 2021
Copy link
Contributor

@fernando-cortez fernando-cortez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For all descriptions: do a pass and add ending punctuation. Rogue sneak ability needs to be more assertive in what it does.

@fernando-cortez fernando-cortez added 2-Reviewed with Comments PR requires owner's attention and removed 1-Needs Review PR needs attention from the assignee and reviewers labels Mar 22, 2021
@eheimburg
Copy link
Contributor Author

Okay, added missing punctuation and revised a few to be accurate.

@fernando-cortez fernando-cortez added 2-One More Review One review in, one to go and removed 2-Reviewed with Comments PR requires owner's attention labels Mar 23, 2021
@pdeschain pdeschain added 3-Good to Merge and removed 2-One More Review One review in, one to go labels Mar 23, 2021
@eheimburg
Copy link
Contributor Author

I'm not authorized to merge in this branch. @fernando-cortez can you please do the honors? Thank you!

@fernando-cortez fernando-cortez merged commit cea2a8a into release/v0.1.0-experimental Mar 24, 2021
@fernando-cortez fernando-cortez deleted the feature/missing-tooltips-for-mage branch March 24, 2021 01:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2-Easy This PR is trivial and can be reviewed quickly 3-Good to Merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants