Labels Milestones

New Pull Request

TRADING YEAH LETS GO #80

Merged

jake merged 41 commits from trading into master 3 years ago

Conversation 0 Commits 41 Files Changed 48

jake commented 3 years ago

Owner

There is no content yet.

jake added 39 commits 3 years ago

571bab2534 print send buffers

0d30df8da6 start of trading

91d381a2e2 this fixed a thing probably

9b78d7ed80 add some Defaults

143ed7ed78 add ItemTransaction struct as begining of item refactor v3

55f82b9fce trace -> info

70a6764360 make these a bit more public for tests

b374b63cba nth(0) -> next()

42eb875f8f split create_item into individual and stacked variants

continuous-integration/drone/push Build is failing

Details

21df9a9340 itemtransaction action takes boxed value

dbb1105741 cleanup

296d1cc0ea InventoryItem::{stacked, mag}

continuous-integration/drone/push Build is failing

Details

6ed0d838d2 add gateway function set_character_meseta

8a29a75397 remove TODO

ac423fa6ba change trade structure in clientstate

continuous-integration/drone/push Build is failing

Details

ddff55d09e add another item pickup test

d80d0fb01d add some functions to Inventory

continuous-integration/drone/push Build is failing

Details

8e82d6c4b4 convert item pickup to new itemtransaction model

98260308e8 stray spaces

continuous-integration/drone/push Build is failing

Details

b80f30ef9d initial actual trading logic

continuous-integration/drone/push Build is failing

Details

82ef5ba2ea RIP ItemLocation

fun while it lasted

ItemLocation ceased to be the canonical place to store an item's
location. replaced with ItemNote which basically covers the actual use
case but without the enforcing of a location.

continuous-integration/drone/push Build is failing

Details

8b87bd8d7b fix mag item data

c90c7c1fd3 update anyhow

d4fc915164 add debug to characterentity

070735edbf ItemDetail::tool

d85ada945e add some functions to Inventory

3865caba8d refcell room id counter

5d410f88f3 PartialEq LocalClientId u8

b3b6dad6ad trades!

continuous-integration/drone/push Build is failing

Details

71ba3c3c3d forgot to add this migration

3b7dcd0295 remove printlns

continuous-integration/drone/push Build is failing

Details

6d2753d082 improve error display a bit

6bd341cd01 check inventory has enough space to accept traded items

continuous-integration/drone/push Build is failing

Details

872794e45f improve error messages

continuous-integration/drone/push Build is failing

Details

3a10de502a tell other players you don't have meseta when you drop it

continuous-integration/drone/push Build is failing

Details

72d72801e1 trade meseta

continuous-integration/drone/push Build is failing

Details

9c34baeb26 meseta refactor

continuous-integration/drone/push Build is failing

Details

ecf1f23c6c fix tests

continuous-integration/drone/push Build is failing

Details

continuous-integration/drone/pr Build is failing

Details

81916d1f57 TRADING JUST IN TIME TO BARELY MISS XMAS

jake added 1 commit 3 years ago

continuous-integration/drone/push Build is failing

Details

continuous-integration/drone/pr Build is failing

Details

22f630f562 properly set item_id after a trade serverside

jake added 1 commit 3 years ago

continuous-integration/drone/push Build is passing

Details

continuous-integration/drone/pr Build is passing

Details

a6e50555de appease the clip

jake merged commit 26a2ef6492 into master 3 years ago

jake referenced this issue from a commit 3 years ago

Merge pull request 'TRADING YEAH LETS GO' (#80) from trading into master

jake deleted branch trading 3 years ago

continuous-integration/drone/push Build is passing

Details

continuous-integration/drone/pr Build is passing

Details

The pull request has been merged as 26a2ef6492.

Sign in to join this conversation.

Reviewers

Request review

No reviewers

Labels

Apply labels

Clear labels

item refactor nice to have one day maybe someday release

No Label item refactor nice to have one day maybe someday release

Milestone

Set milestone

Clear milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Assign users

Clear assignees

No Assignees

1 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Write Preview

Loading…

Cancel

Save

There is no content yet.