make cookie module shared #789

Merged
merged 2 commits into from Sep 9, 2014

Projects

None yet

7 participants

@LennyPenny
Contributor

I don't see why it should be client only
Would also be nice to have this easy to use abstraction layer on the server

test:
img

LennyPenny added some commits Aug 18, 2014
@LennyPenny LennyPenny make cookie module shared
I don't see why it should be client only, and it would be nice to have this easy to use abstraction layer on the server

test:
http://a.pomf.se/bphqgs.png
8a0e32f
@LennyPenny LennyPenny Update init.lua 9b7ba19
@ThatLing

+1 need cookie module

@Capster
Capster commented Aug 18, 2014

send.txt please

@LennyPenny
Contributor

What

@LennyPenny
Contributor

It's already on the client? I just made it serverside too
sdsd

@Capster
Capster commented Aug 18, 2014

Serverside, lol. Sorry :)

@Jake-TD
Jake-TD commented Aug 18, 2014

+1 for cookies

@8BallBomBom

Must have. +1

@willox
Collaborator
willox commented Aug 20, 2014

Change the console commands to not work serverside if a client tries to call them. Perhaps give them sv/cl suffixes too.

@LennyPenny
Contributor

Well looks like you can't print any other cookies than menu ones via the client anyway (menu command gets added first, and client/server can't override the concommand)
Fixing cookies itself is up for another pull req though

@robotboy655 robotboy655 merged commit 908f2c5 into garrynewman:master Sep 9, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment