search

qbcore-framework / qb-weathersync

Weather System Used With QB-Core :sun_behind_rain_cloud:
GNU General Public License v3.0
23 stars 142 forks source link

[ENHANCEMENT] #63 - Implement real time synchronisation #74

Closed JensGryspeert closed 6 months ago

JensGryspeert commented 1 year ago

Describe Pull request This pull request includes several commits that aim to enhance the time synchronization and network clock time updating functionality in the project which there is now an option to activate real time synchronization.

The specific changes made are as follows:

  1. Refactor time synchronization and network clock time updating

    • Determined the time increment based on the RealTimeSync configuration setting.
    • Used the GetLocalTime() function to retrieve the local time in RealTimeSync mode.

  2. Implement real-time synchronization server-side

    • Implemented real-time synchronization functionality on the server-side.
    • Ensured that the server time is set based on the actual real-time retrieved from an API.

  3. Implement function to fetch current Unix time from API

    • Added a function to retrieve the current Unix time from an API.
    • Used the function to set the server time in the real-time synchronization process.

  4. Add variable in config to activate real-time sync

    • Added a configuration variable to enable/disable real-time synchronization.

Questions (please complete the following information):

github-actions[bot] commented 10 months ago

This PR has had 60 days of inactivity & will close within 7 days

JensGryspeert commented 10 months ago

Can someone review this? Just retrieved an message in mail the following:

This PR has had 60 days of inactivity & will close within 7 days
GhzGarage commented 7 months ago

@JensGryspeert Could you fix the conflict if possible please

JensGryspeert commented 7 months ago

I will, hope the PR will be merged then. Otherwise the conflict didn't appeared :). I'll do this this evening.

citRaTTV commented 7 months ago

I will, hope the PR will be merged then. Otherwise the conflict didn't appeared :). I'll do this this evening.

@JensGryspeert still seeing this conflict. Have you had a chance to sync your branch with latest & resolve it?

JensGryspeert commented 7 months ago

Not yet. Forgot it to be honest. Will do this tomorrow. I've planned it in my calendar.

JensGryspeert commented 6 months ago

I will, hope the PR will be merged then. Otherwise the conflict didn't appeared :). I'll do this this evening.

@JensGryspeert still seeing this conflict. Have you had a chance to sync your branch with latest & resolve it?

Rebased the branch and resolved it ;)