Skip to content

docs(applications): Feature Guide for Applications #14178

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

gabe-lyons
Copy link
Contributor

Assets already added to static assets.

Copy link

alwaysmeticulous bot commented Jul 22, 2025

🔴 Meticulous spotted visual differences in 19 of 1543 screens tested: view and approve differences detected.

Meticulous evaluated ~11 hours of user flows against your PR.

Last updated for commit ac2b9c2. This comment will update as new commits are pushed.

@datahub-cyborg datahub-cyborg bot added the needs-review Label for PRs that need review from a maintainer. label Jul 22, 2025
Copy link

codecov bot commented Jul 22, 2025

Bundle Report

Changes will decrease total bundle size by 26.09kB (-0.12%) ⬇️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
datahub-react-web-esm 22.25MB -26.09kB (-0.12%) ⬇️

Affected Assets, Files, and Routes:

view changes for bundle: datahub-react-web-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-*.js -26.09kB 18.61MB -0.14%

Copy link

codecov bot commented Jul 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@github-actions github-actions bot requested a deployment to datahub-wheels (Preview) July 28, 2025 17:42 Abandoned

This guide will show you how to

- Create an application.
Copy link
Collaborator

Choose a reason for hiding this comment

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

@gabe-lyons should we also include fetching an application? It was in the feature guide but not here

query getApplication {
  application(urn: "urn:li:application:engineering") {
    urn
    properties {
      name
      description
    }
    children {
      total
    }
  }
}

@datahub-cyborg datahub-cyborg bot added pending-submitter-response Issue/request has been reviewed but requires a response from the submitter and removed needs-review Label for PRs that need review from a maintainer. labels Aug 9, 2025
@datahub-cyborg datahub-cyborg bot added needs-review Label for PRs that need review from a maintainer. and removed pending-submitter-response Issue/request has been reviewed but requires a response from the submitter labels Aug 9, 2025
Copy link
Collaborator

@maggiehays maggiehays left a comment

Choose a reason for hiding this comment

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

@gabe-lyons this is good to go from my side; will defer to you if you want to include the URN lookup example in the API guide or not

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Issues and Improvements to docs pending-submitter-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants