Skip to content

[workday][activity] Add Activity datastream#19525

Open
muskan-agarwal26 wants to merge 1 commit into
mainfrom
feature/workday-0.1.0
Open

[workday][activity] Add Activity datastream#19525
muskan-agarwal26 wants to merge 1 commit into
mainfrom
feature/workday-0.1.0

Conversation

@muskan-agarwal26

Copy link
Copy Markdown
Contributor

Proposed commit message

This integration collects data from activity log data stream in the Workday platform. Workday fields are mapped to their corresponding ECS fields where possible.

Test samples were derived from live data samples, which were subsequently sanitized.

API documentation: https://community.workday.com/sites/default/files/file-hosting/restapi/index.html#privacy/v1/activityLogging

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Related issues

The release includes activity data stream and associated dashboard.

Workday fields are mapped to their corresponding ECS fields where possible.

Test samples were derived from live data samples, which were subsequently
sanitized.

---------

Co-authored-by: muskan-crest <muskan.agarwal@crestdata.ai>
@muskan-agarwal26 muskan-agarwal26 requested a review from a team as a code owner June 15, 2026 13:28
@muskan-agarwal26 muskan-agarwal26 changed the title [workday][activity] Add Activity datastream (#19319) [workday][activity] Add Activity datastream Jun 15, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Elastic Docs Style Checker (Vale)

Summary: 1 warning, 1 suggestion found

⚠️ Warnings (1): Fix when the suggestion improves clarity or correctness.
File Line Rule Message
packages/workday/_dev/build/docs/README.md 5 Elastic.EndPuntuaction Don't end headings with punctuation.
💡 Suggestions (1): Optional style improvements. Apply when helpful.
File Line Rule Message
packages/workday/changelog.yml 1 Elastic.Versions Use 'later versions' instead of 'newer versions' when referring to versions.

The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale.

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown

Changelog link mismatch — expected https://github.com/elastic/integrations/pull/19525 in the following file(s):

  • packages/workday/changelog.yml

Tip

If expected, add the changelog-link-check:skip label to skip this check. Or, if an issue link was intended, use .../issues/<n> instead.

View Buildkite build
@muskan-agarwal26

@andrewkroh andrewkroh added New Integration Issue or pull request for creating a new integration package. documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Crest Contributions from Crest developement team. labels Jun 15, 2026
@efd6

efd6 commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

/test

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History


## Overview

[Workday](https://www.workday.com/en-in/homepage.html) is a cloud-based ERP system that manages business processes and allows organizations to use an integrated application. Workday is a coherent cloud ERP system for financial analysis, analytical solutions, HCM suites, and better business processes.

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.

This vale lint warning doesn't make sense; it's complaining about a paragraph on the basis that the text is a heading, but it's not.

cc @theletterf

Comment on lines +86 to +90
"cursor": {
?"last_timestamp": state.?cursor.last_timestamp,
?"max_ingested_time": state.?cursor.max_ingested_time,
"access_token": "",
},

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.

In the case of an object state.events, the cursor is ignored, so this is an ineffectual assignment (ref).

Comment on lines +163 to +167
"cursor": {
?"last_timestamp": state.?cursor.last_timestamp,
?"max_ingested_time": state.?cursor.max_ingested_time,
"access_token": "",
},

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.

Same

Comment on lines +185 to +189
"cursor": {
?"last_timestamp": state.?cursor.last_timestamp,
?"max_ingested_time": state.?cursor.max_ingested_time,
"access_token": tok.access_token,
},

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.

And here

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

Labels

changelog-link-check:skip Crest Contributions from Crest developement team. documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. New Integration Issue or pull request for creating a new integration package.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants