Velocity icon indicating copy to clipboard operation
Velocity copied to clipboard

Published 20 hours ago • verified publisher icon PaperMC

Transfer applied resource packs

Open Gerrygames opened this issue 10 months ago • 2 comments

Gerrygames avatar Jan 17 '25 11:01 Gerrygames

Is it worth making this a generic record storage in case additional fields could be transferred in the future? For now though it may just be something like:

record TransferSession(Collection<ResourcePackInfo> appliedPacks) {}

May just make it easier to add onto in the future.

Owen1212055 avatar Jan 17 '25 13:01 Owen1212055

The size of a cookie is fairly limited, if we want to store additional info it would make more sense to send additional cookies, IMHO; I guess you could make it more abstracted out, but, we'd still be sending separate cookies out, etc; so, I'm not 100% sure entirely what we'd get trying to combine stuff too much (other than a bunch more code to maintain)

electronicboy avatar Jan 17 '25 13:01 electronicboy