search

amtgard / ORK3

Version 3 of the Online Record Keeper
Other
23 stars 11 forks source link

Cannot revoke or add dues via player update #219

Closed kenwalker closed 5 years ago

kenwalker commented 5 years ago

Closes amtgard/ORK3#218

Add module load function, and load Treasury to revoke dues, returning the result

kenwalker commented 5 years ago

This is required for my edit player page in my mobile application.

kenwalker commented 5 years ago

I'm updating this PR to include adding dues. Will comment when done.

kenwalker commented 5 years ago

Ok, I've added both in and the app currently works to revoke or add dues.

kenwalker commented 5 years ago

Waivered in PlayerUpdate was checking for true, the CreatePlayer path looks for 1 or 0. Updated to honor that in player update

See https://github.com/amtgard/ORK3/blob/master/system/lib/ork3/class.Player.php#L394