Skip to content

(Chat): Adding an option to register suggestions server side. - #148

Open
spoty123 wants to merge 1 commit into
citizenfx:masterfrom
spoty123:master
Open

(Chat): Adding an option to register suggestions server side.#148
spoty123 wants to merge 1 commit into
citizenfx:masterfrom
spoty123:master

Conversation

@spoty123

Copy link
Copy Markdown

The main purpose of this pull request is to allow registering suggestions before clients have joined the server, and let them know about those suggestions once they join.
From what I understood most "frameworks" basically save all of the server-side suggestions on the server until the client joined and only then registering those suggestions from the client itself, I believe that this will serve as a better solution (which doesn't interrupt the existing one), added a convar in case people want to remove that.

The main purpose of this pull request is to allow registering suggestions before clients have joined the server, and let them know about those suggestions once they join.
From what I understood most "frameworks" basically save all of the server-side suggestions on the server until the client joined and only then registering those suggestions from the client itself, I believe that this will serve as a better solution (which doesn't interrupt the existing one), added a convar in case people want to remove that.
@flammensturm

Copy link
Copy Markdown

vouch this

@jaymo1011

Copy link
Copy Markdown
Contributor

This may be a month old but I guess no one else has brought this up, will this work with #101? It may be better to wait for the chat rework to be completed or better yet, make this as a suggestion to the chat rework itself!

@spoty123

Copy link
Copy Markdown
Author

make this as a suggestion to the chat rework itself!

I didn't even know about this lol, this is what I'm going to do right now.

@spoty123 spoty123 closed this Feb 25, 2021
@spoty123 spoty123 mentioned this pull request Feb 25, 2021
@spoty123

Copy link
Copy Markdown
Author

Reopening this since #101 was already merged?

@spoty123 spoty123 reopened this Apr 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants