JKorf / Bybit.Net

A C# .netstandard client library for the Bybit REST and Websocket V5 API focusing on clear usage and models
https://jkorf.github.io/Bybit.Net/
92 stars 64 forks source link

Fix Bug in Demo Trading: Update Server Time Endpoint to API v5 #206

Closed Amitabitbul closed 5 months ago

Amitabitbul commented 5 months ago

This pull request addresses a bug in the Demo Trading system where the endpoint to retrieve the Server Time was using the outdated v3 API. Since the Demo Trading API has shifted to version 5, the endpoint has been updated to ensure compatibility and proper functioning.

Changes Made

This change should resolve any issues with retrieving the Server Time and align the Demo Trading API with the current version. Let me know if any additional tests or reviews are needed.

JKorf commented 5 months ago

Hi, I already had a fix for this in my local changes, so this PR is not needed anymore as it's already fixed in the latest version. I see you also updated the derivatives GetServerTimeAsync endpoint to V5. I didn't include this as the demo environment only supports the V5 API, and the derivatives API is V3.