Skip to content

[CXE-3] Add additional parameters to application payload in README#85

Merged
hisanibrahim merged 1 commit intomasterfrom
CXE-3
Apr 23, 2026
Merged

[CXE-3] Add additional parameters to application payload in README#85
hisanibrahim merged 1 commit intomasterfrom
CXE-3

Conversation

@hisanibrahim
Copy link
Copy Markdown
Member

@hisanibrahim hisanibrahim commented Apr 6, 2026

Changes

Adds timezone, userAgent, acceptLanguage, and referer to the apply endpoint field table in the README, noting that the header-based fields fall back to request headers if omitted (relevant for backend/proxy callers forwarding browser values).

Links

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the public API documentation for the “Apply to a job posting” endpoint by documenting additional optional fields that can be included in an application payload.

Changes:

  • Documented a timezone field for applicant timezone context.
  • Documented userAgent, acceptLanguage, and referer fields, including header-derived defaults and truncation limits.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@hisanibrahim hisanibrahim marked this pull request as ready for review April 7, 2026 09:08
@hisanibrahim hisanibrahim requested a review from cjairm April 7, 2026 10:18
Copy link
Copy Markdown

@jaelee213 jaelee213 left a comment

Choose a reason for hiding this comment

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

Code/docs content looks good and backwards compatible.

I’m holding approval only because the dependency-review check is currently failing in this PR. Please rerun/fix that check; once green this should be ready from a code-review perspective.

Copy link
Copy Markdown

@jaelee213 jaelee213 left a comment

Choose a reason for hiding this comment

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

changes lgtm

@hisanibrahim hisanibrahim merged commit f61aac5 into master Apr 23, 2026
4 of 5 checks passed
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