Skip to content

Add 'user_reason' to MyoController for postTransfer.#64

Merged
itinerare merged 2 commits into
lk-arpg:developfrom
preimpression:develop
Jan 11, 2021
Merged

Add 'user_reason' to MyoController for postTransfer.#64
itinerare merged 2 commits into
lk-arpg:developfrom
preimpression:develop

Conversation

@preimpression
Copy link
Copy Markdown
Contributor

Add 'user_reason' to MyoController for postTransfer.

Previously, when attempting to user-transfer a MYO slot, it would give a "undefined index: user_reason" error.

Copy link
Copy Markdown
Collaborator

@Draginraptor Draginraptor left a comment

Choose a reason for hiding this comment

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

Just a tiny format thing, but maybe add a space after the comma?

Add space before 'user_reason'
@itinerare itinerare added the needs review Pull requests that are pending community review label Jan 11, 2021
@itinerare itinerare added reviewed Pull requests that have received community review and are pending merge and removed needs review Pull requests that are pending community review labels Jan 11, 2021
@itinerare itinerare merged commit a3b5e68 into lk-arpg:develop Jan 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

reviewed Pull requests that have received community review and are pending merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants