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 github_tag table to use GraphQL API #238

Closed
graza-io opened this issue May 3, 2023 · 3 comments
Closed

Update github_tag table to use GraphQL API #238

graza-io opened this issue May 3, 2023 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@graza-io
Copy link
Contributor

graza-io commented May 3, 2023

Is your feature request related to a problem? Please describe.
I'd like to use the GraphQL API when querying the table

Describe the solution you'd like
Update the table to use the GraphQL API

Describe alternatives you've considered
N/A

Additional context

For any fields that have paging, consider implementing that field as another table to avoid excess API calls (as per https://steampipe.io/docs/develop/plugin-release-checklist#data-ingestion).

New columns

  • tagger_date - date the tag was created if available
  • tagger_name - name of the user whom created the tag. (null if custom tag_message set)
  • message - message associated with the tag.
  • commit - JSON column for nested commit

Removed columns

  • zipball_url - embedded in commit
  • tarball_url - embedded in commit
  • commit_sha - embedded in commit
  • commit_url - embedded in commit

Columns with data changes (please include examples of old and new data for each entry)

  • All urls now point to web app rather than REST API endpoints.
@graza-io graza-io added the enhancement New feature or request label May 3, 2023
@graza-io graza-io self-assigned this May 3, 2023
@graza-io graza-io linked a pull request May 3, 2023 that will close this issue
@graza-io
Copy link
Contributor Author

graza-io commented May 11, 2023

Commit: e9368e0 on graphql-dev branch.

Edit: this commit had all fields for commit as separate columns in violation with guidelines, will be reworked.

@graza-io
Copy link
Contributor Author

graza-io commented May 11, 2023

Commit: c5997e9 on graphql-dev branch.

@cbruno10
Copy link
Contributor

cbruno10 commented Jun 16, 2023

Completed in 42680ef

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants