Skip to content
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

Update docs based on code PR #1 #8

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

Conversation

runllm-pr-agent[bot]
Copy link

Changes Made

  • docs/_api-guide-template.md
    • # [Feature Name] --> # Include Project and Model Names i...: The code change introduces a new feature that adds project and model names to the custom properties of Looker explores. The documentation should reflect this new feature by updating the feature name to describe the new functionality.
    • ### Goal of This GuideThis guide wi... --> ### Goal of This GuideThis guide wi...: The goal of the guide should be updated to reflect the new feature, which is to include project and model names in Looker explores' custom properties.
    • ## [Action] [Feature Name] --> ## Add Project and Model Names to L...: The action section should be updated to describe the specific action related to the new feature, which is adding project and model names to Looker explores.
  • docs/api/tutorials/custom-properties.md
    • In this example, we will add some c... --> In this example, we will add some c...: The code changes introduced new custom properties 'project' and 'model' to the Looker dataset properties. The documentation should reflect these additions to provide accurate guidance on how to use custom properties.
    • The following code adds custom prop... --> The following code adds custom prop...: The documentation should be updated to include the new custom properties 'project' and 'model' that were added in the code changes, ensuring that users are aware of all the properties they can add.
    • You can now see the two new propert... --> You can now see the four new proper...: The documentation should accurately reflect the number of new properties added, which now includes 'project' and 'model' in addition to 'cluster_name' and 'retention_time'.
  • docs/modeling/metadata-model.md
    • This file associates dataset-specif... --> This file associates dataset-specif...: The code changes added 'project' and 'model' properties to the custom properties of Looker explores, which are part of the DatasetProperties aspect. The documentation should reflect these new properties to provide a complete and accurate description of the DatasetProperties aspect.
  • docs/quick-ingestion-guides/looker/overview.md
    • Looker connector will ingest Looker... --> Looker connector will ingest Looker...: The code changes introduce new properties 'project' and 'model' to the Looker metadata ingestion process. The documentation should be updated to reflect these new properties, ensuring users are aware of the additional metadata that can be ingested.
  • docs/quick-ingestion-guides/looker/configuration.md
    • Enter the details into the Looker R... --> Enter the details into the Looker R...: The code changes added project and model names to the custom properties of Looker models and explores. The documentation should be updated to inform users that they can now specify these properties in the Looker ingestion configuration.
  • docs/api/restli/restli-overview.md
    • ##### Browse an EntityTo browse (ex... --> ##### Browse an EntityTo browse (ex...: The code changes introduce new custom properties 'project' and 'model' for Looker entities. The documentation should mention these new properties to inform users that they can now query or browse Looker entities with these additional metadata attributes.
    • ##### Search an EntityTo search for... --> ##### Search an EntityTo search for...: The documentation should be updated to inform users that they can now search for Looker entities using the newly added custom properties 'project' and 'model'. This will help users leverage the full capabilities of the updated API.

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.

0 participants