Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Whitelist function #43

Open
lorenzo93 opened this issue Feb 7, 2024 · 12 comments
Open

Whitelist function #43

lorenzo93 opened this issue Feb 7, 2024 · 12 comments

Comments

@lorenzo93
Copy link
Contributor

Hi,

I'm Italian and I speak only english (except italian). I've translated your readme using a translator so I'm not sure if I got everything correct.
Is there in your software a whitelist function?
Is there support for translations? I'm a C# developer and I could help :)

@KirosHan
Copy link
Owner

KirosHan commented Feb 8, 2024

Hi @lorenzo93
thanks for your attention,i am sorry there is no whitelist function now
but the localization work for the program is already in the development plan
It would be great if you could give me a hand

@lorenzo93
Copy link
Contributor Author

lorenzo93 commented Feb 8, 2024

I read the code today. Not in depth but I understand how it is programmed.
I have some questions:

  1. Do you have your development plan public somewhere?
  2. I'm having some difficoulties opening some of the files because of the encoding., the chinese characters (mainly comments) are not correctly recognised, so if I save the file to make a change it gets corrupted. Which encoding do you use?
  3. If I make some changes to the structure of the project, are you willing to accept a PR? I would want to organize the code in folders/namespaces and extract all the application login from the form so that it could be easier to mantain.
  4. Once point 2 is completed, my main need is to have the whitelist support, so I'm going to implement that part. Is that ok for you to add this functionality?

P.S.: I've started working, you can find the project in my profile (I've forked yours). Let me know what you think :)

@KirosHan
Copy link
Owner

KirosHan commented Feb 9, 2024

Firstly, the recent work plan has been somewhat stagnant due to a lack of creative ideas, so there is no additional specific development plan for now.
Later on, I will check the encoding, but most of the encoding should be UTF-8.
Also, I will accept beneficial all Pull requests including the whitelist function

For users, the operation should be as simple as possible.

@KirosHan
Copy link
Owner

KirosHan commented Feb 9, 2024

Can you tell me which files you encounter problems with when opening?

@lorenzo93
Copy link
Contributor Author

I had problem with the Form1.cs, but I fixed it opening with chinese encoding and than saving it again in UTF-8.
I made a first working version of the Whitelist function, I've refactored all the communication classes (RCON, PalUserInfo, etc).
I have also implemented automatic settings reload if disk file changes.
If you want, check the code at my repository.

@lorenzo93
Copy link
Contributor Author

I modified your code to use the RCON Client I made. I also changed the timers to be on separate threads so if they hang for some reason the app doesn't freeze.
Try to compile and test my code, I have probably messed up some chinese strings that I just copied 😄

@KirosHan
Copy link
Owner

Your code is great!
And you can submit a PR when your work is done.If there are no issues, I will accept it.
Thanks for your contributions. XD

@KirosHan
Copy link
Owner

Hi @lorenzo93
I have accepted your PR and did a simple test, but I found that most of the Rcon functions do not work properly. Currently, only “Broadcast” command can be sent normally, but the echo also shows as sending failed.

@lorenzo93
Copy link
Contributor Author

Hi,
I've forgot to commit some lines.
Now it is working in #47

@KirosHan
Copy link
Owner

that's okay
it is worked😄

@lorenzo93
Copy link
Contributor Author

great!

@KirosHan
Copy link
Owner

hi @lorenzo93
I have completed some UI localization work. If you're willing, could you help me confirm whether the translations are correct?

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

No branches or pull requests

2 participants