Skip to content

Conversation

@Asheraf
Copy link
Contributor

@Asheraf Asheraf commented Nov 5, 2017

Pull Request Prelude

Affected Branches:
master

Known Issues and TODO List

non

@ghost ghost assigned Asheraf Nov 5, 2017
@ghost ghost added the status:inprogress Issue is being worked on / the pull request is still a WIP label Nov 5, 2017
@HerculesWSAPI
Copy link
Contributor

This change is Reviewable

@Asheraf Asheraf added status:code-review Awaiting code review and removed status:inprogress Issue is being worked on / the pull request is still a WIP labels Nov 5, 2017
@Asheraf Asheraf force-pushed the homunaf branch 2 times, most recently from f435057 to 2ea00d5 Compare November 5, 2017 02:21
Copy link
Contributor

@4144 4144 left a comment

Choose a reason for hiding this comment

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

Please update also main.sql with new field and updates list

@Asheraf Asheraf force-pushed the homunaf branch 2 times, most recently from 7a8c0d8 to 5283e5e Compare November 5, 2017 20:20
@Asheraf Asheraf added this to the Release v2017.11.19 milestone Nov 12, 2017
@Asheraf Asheraf added the type:enhancement Issue describes an enhancement or feature that should be implemented label Nov 12, 2017
@MishimaHaruna
Copy link
Member

What do you think about adding a battle_config pref to optionally disable this (it's fine if it's enabled by default)?
I can see how adding this feature can be somehow controversial, and some might want to make it unavailable.

@Asheraf Asheraf force-pushed the homunaf branch 2 times, most recently from 27503b7 to e38cb92 Compare November 18, 2017 19:44
@Asheraf
Copy link
Contributor Author

Asheraf commented Nov 18, 2017

@MishimaHaruna done.

@MishimaHaruna MishimaHaruna modified the milestones: Release v2017.11.19, future release, Release v2017.12.17 Nov 19, 2017
@MishimaHaruna MishimaHaruna merged commit 85b71be into HerculesWS:master Dec 8, 2017
@ghost ghost removed the status:code-review Awaiting code review label Dec 8, 2017
@Asheraf Asheraf deleted the homunaf branch December 9, 2017 18:27
Lemongrass3110 added a commit to rathena/rathena that referenced this pull request Mar 26, 2018
This feature is available in 2017-09-20bRagexeRE or later

Credits to @Asheraf in HerculesWS/Hercules#1898
secretdataz pushed a commit to rathena/rathena that referenced this pull request Apr 4, 2018
* Added support for homunculus autofeeding
This feature is available in 2017-09-20bRagexeRE or later

Credits to @Asheraf in HerculesWS/Hercules#1898
CrazyBS pushed a commit to CrazyBS/rathena that referenced this pull request Apr 16, 2018
* Added support for homunculus autofeeding
This feature is available in 2017-09-20bRagexeRE or later

Credits to @Asheraf in HerculesWS/Hercules#1898
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:enhancement Issue describes an enhancement or feature that should be implemented

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants