Add function and script that return IDs of open positions#289
Add function and script that return IDs of open positions#289illia-malachyn merged 5 commits intov0from
Conversation
17e1ed9 to
b929229
Compare
b929229 to
65bd300
Compare
|
I've just understood that it might be better to write a script that fetches a number of positions with an offset. Sth like: This is doable and it's a faster way to retrieve positions. I'll update PR with a new script |
|
I think with such a script it will be easier to manage positions #294 |
|
@illia-malachyn can you also create a similar PR to the main branch and link it here to keep track of backported features |
jordanschalm
left a comment
There was a problem hiding this comment.
I prefer this solution over #294. I feel like dealing with both position ID and index is a bit confusing.
If we pair this with a script to batch-retrieve position details by list of position IDs (to avoid needing one script call per position), I think we'll have what we need.
UlyanaAndrukhiv
left a comment
There was a problem hiding this comment.
I like the current approach, think it is better cause returns all position ids in a single call.
Exactly! That is what I was thinking of as the 3rd approach. I'll add such a script to this PR |
Adding a script to get the IDs of open positions. FCM observer needs this information to query the position health factors.
Update: backported to main #296