Suggestion Default Language in the Constructor

[Igor Dot Ninja]

I didnt want to submit an issue, because i dont know if its by design.

 

By default the language for the steam-tradeoffer-manager is not set

 

If somebody wants to set it, they need to set it up manually

 

var SteamTradeOfferManager = require("steam-tradeoffer-manager");

manager =   new SteamTradeOfferManager({"language": "en"})

 

----

 

When the language is not set, some properties of the trade offer are not accurate. 

 

itemsToGive and itemsToReceive

 

show the correct appid, contextid, assetid and classid which is important

 

however marketable, commodity and market restrictions are showing wrong info, all those are false (0).

 

i understand that setting up language will show different descriptions and names in some regions. and by not setting the language, you dont get extra info, which saves bandwidth, and speeds up things.

 

So maybe its useful to set the language to "en" by default.

[Dr. McKay]

It's intentional. If you want names and such, you need to set a language. If you don't need them, don't set it. Not setting a language saves resources and memory as information for the given language needs to be downloaded from Steam separately and cached in memory.

[Igor Dot Ninja]

Thank you for clearing that up