Skip to content

feat(UNTIL-21674): stocks report as of query param - #763

Merged
bogdanmbpfa merged 2 commits into
masterfrom
feature/UNTIL-21674-stocks-report-as-of-query-param
Aug 24, 2026
Merged

feat(UNTIL-21674): stocks report as of query param#763
bogdanmbpfa merged 2 commits into
masterfrom
feature/UNTIL-21674-stocks-report-as-of-query-param

Conversation

@bogdanmbpfa

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds support for passing an as_of timestamp (and related date range fields) through the v2 analytics “stocks” report request options so consumers can request the report “as of” a specific time.

Changes:

  • Extended StocksExportOptions to include as_of, start, and end query parameters.
  • Added a unit test ensuring as_of is serialized into the request query string for the stocks report.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
test/analytics_v2/stocks/index.test.ts Adds coverage to verify as_of is forwarded into the stocks report request URL query string.
src/v2/analytics/reports/stocks.ts Extends the public options type to allow as_of/start/end query params for stocks report requests.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@bogdanmbpfa
bogdanmbpfa merged commit 0967e52 into master Aug 24, 2026
1 check passed
@bogdanmbpfa
bogdanmbpfa deleted the feature/UNTIL-21674-stocks-report-as-of-query-param branch August 24, 2026 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants