Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add stats#125

Open
ILYAGVC wants to merge 14 commits intoMixV2:masterfrom
ILYAGVC:master
Open

Add stats#125
ILYAGVC wants to merge 14 commits intoMixV2:masterfrom
ILYAGVC:master

Conversation

@ILYAGVC
Copy link
Contributor

No description provided.

@@ -0,0 +1,27 @@
## User Collection Stats (BULK)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

i think i would merge this with the regular stats bulk endpoint, withcategory as an optional query param


``startDate``: 0 (for alltime) or set for custom time-window

``endDate``: 9223372036854775807 (for alltime) or set for custom time-window

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

wouldn't leaving out theendDate mean alltime?

i think it should also be documented what format those numbers have

| - | - | - |
| https://statsproxy-public-service-live.ol.epicgames.com/statsproxy/api/statsv2/account/:accountId | `GET` | Yes |

## Parameters

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

startDate andendDate are missing here

"startDate": 0,
"endDate": 9223372036854775807,
"owners": ["accountId"],
"stats": ["s22_social_bp_level"]

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

stats could be any stats key. you should probably put in a placeholder value and document it below

}
```

## Parameters

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

startDate andendDate should be documented aswell

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

1 more reviewer

@ThisNilsThisNilsThisNils requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ILYAGVC@ThisNils

Comments


[8]ページ先頭

©2009-2026 Movatter.jp