New Pull Request

roominfo #112

Merged

jake merged 61 commits from roominfo into master 2 years ago

andy commented 2 years ago

Collaborator

There is no content yet.

andy added 4 commits 2 years ago

e349a41b16 room tab info page 1

4202ef8ff7 handle rooms that close after downloading the room list in the lobby

74e54935ad thanks clippy you are the best

9638181261 'u up?' tests for rooms

andy added 1 commit 2 years ago

9417105359 giving this character a name to prove i am a wise and omnipotent diety

andy added 55 commits 2 years ago

4e1c6300d4 update sqlx

a2e78014ee make all entitygateway functions take &mut self, move postgres functions to separate function for future use

cbb5c1fffd add transactions!

bc702375d2 redo this cause entitygateway isnt Clone anymore

e337852d72 initial ItemState

3c8a35de83 remove unused use

bdfcb5c0ad redundant bit of code

7aa4a6394f cleanup

da066a3445 move EntityGatewayTransaction to end of file

3d51b3ee57 don't need static for with_transaction

e58aff7042 remove boxes from ItemStateAction structs to remove need for static lifetimes

916bb31671 move item state action functions to new file

da0c75fa39 cleanup

5d6f9deb59 mostly appease clippy

eac0a31056 cool I can simplify this now

1eae3795af add character_id to ItemNote::PlayerDrop

b8de2ea8e6 add item_state to ShipState

acdf960d4b add/move stuff around for itemstate

810d8269af move drop item over to item_state

8c403f0d0e missed fixing these

bd6b8effcd reorder function

9a2d1945af I was wondering why nightly a week ago was letting me get away with this

646f75d00d move partial drops over to item_state

1f7086396e wtf happened here

bd35481c28 the start of a long process of replacing ItemManager

94c50c011d entitygateway stuff should take &BankName not BankName

4ccdf579bd make inmemorytransaction actually work

03df819a04 bank itemstate stuff

7807cf2b27 spacing

cf09433df2 equipping!

b473af7680 I am dumb

95195d07e4 sort inventory

6306120692 spacing

657dabca27 fix item pickup tests

4e88f684c5 bare minimum item usage stuff

5be796cb5c clean these functions up a bit

d3d8d766b8 magstuff

eb9fdb485b buy items from shop

6bacdd4fe3 selling items

5f4325429e trade refactor

2775f81cc9 fix some meseta-taking things

ff072ca026 enemy drop refactor

b0b34b6616 add postgres trade stuff

9c878ed484 refactor tekking

5976030970 remove old item manager code

fae1ffe55b appease clippy, the tyrant

2771f5692a missed a thing

8f72c6074b move ItemStateAction to its own file

3babc6f4bd move inventory code out of state

be5c3ff633 move floor code out of state

919d509c3a move bank code out of state

286b728441 split actions into actions+tasks

0cd92fd4ac put proper error strings on this struct

b80e308cf2 room tab info page 1

ee8c80f42f rebasing onto the last refactor ever

andy added 1 commit 2 years ago

95b0760e6a Merge branch 'master' into roominfo

jake reviewed 2 years ago

src/ship/packet/handler/lobby.rs

												
					            let gc = cs.user.guildcard;

					            let name = &cs.character.name;

					            let cc = cs.character.char_class;

					            let leveltable = CharacterLevelTable::default();

jake commented 2 years ago

Poster

Owner

wow is it really necessary to construct this here? out of scope for this pr but this should probably be some sort of global const

jake merged commit 7219481aa2 into master 2 years ago

jake referenced this issue from a commit 2 years ago

Merge pull request 'roominfo' (#112) from roominfo into master

jake deleted branch roominfo 2 years ago

andy referenced this pull request 2 years ago

wow is it really necessary to construct this here? out of scope for this pr but this should probably be some sort of global const #114

continuous-integration/drone/pr Build is passing

Details

continuous-integration/drone/push Build is passing

Details

The pull request has been merged as 7219481aa2.

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No project

No Assignees

2 Participants

Notifications

Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.