Skip to content

Added DEL key action for gaining 100 exp in battles#1

Open
Cyan627 wants to merge 1 commit into
aldelaro5:masterfrom
Cyan627:master
Open

Added DEL key action for gaining 100 exp in battles#1
Cyan627 wants to merge 1 commit into
aldelaro5:masterfrom
Cyan627:master

Conversation

@Cyan627
Copy link
Copy Markdown

@Cyan627 Cyan627 commented Jul 12, 2020

Hello to everyone who sees this,

I added one new functionality to the Practice DLL so that it can add 100 Exploration Points to any battle by pressing the DEL key in battle (the UI bugs out slightly because they show up as small and not big exp orbs but it's functional for the time being until I learn how to fix that).

Aside from that, I changed the version numbers and added my name which I understand this isn't anything groundbreaking so I'm fine with all other changes being removed. I do plan on working on this more in the future though.

This is also my first time working with C# and really my first contribution to any open source project so I wanted to give a huge shoutout and thank you to aldelaro5 for giving me the basics and basically carrying me through this first pull request.

Copy link
Copy Markdown
Owner

@aldelaro5 aldelaro5 left a comment

Choose a reason for hiding this comment

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

Other than that, the code seems good to me, however, I will need to test this and it seems I will be busy the coming week, but I should have time to properly test and review this next week. In the meantime, feel free to do the requested changes :)

Comment thread SpeedrunPracticePlugin.cs Outdated
namespace SpeedrunPractice
{
[BepInPlugin("com.aldelaro5.BugFables.plugins.SpeedrunPractice", "Speedrun Practice", "2.0.1")]
[BepInPlugin("com.aldelaro5.BugFables.plugins.SpeedrunPractice", "Speedrun Practice", "2.0.2")]
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Don't version bump as part of the commit, I will be the one to do it when I do a release to tag it.

@aldelaro5 aldelaro5 changed the title Added DEL key action for battle Added DEL key action for gaining 100 exp in battles Jul 12, 2020
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