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

Graph API bug fixes. #1

Conversation

timadair
Copy link
Contributor

Null pointers
Wrong query param annotation
Incorrect conditionals
Redundant initializers
Unused values
Used StringBuilder in one place instead of frequent String +=.

Null pointers
Wrong query param annotation
Incorrect conditionals
Redundant initializers
Unused values
Used StringBuilder in one place instead of frequent String +=.
@barrycaceres barrycaceres merged commit 20075a8 into senzing-garage:issue-1.caceres.initial Dec 28, 2018
@barrycaceres
Copy link
Contributor

Everything looked fine Tim. I should have tested some of the obscure parameters. Also wondering if buildOut default value should be zero.

@timadair timadair deleted the graph-api-cleanup branch January 7, 2019 22:50
This was referenced Nov 18, 2019
@docktermj
Copy link
Contributor

Issue number: #1

barrycaceres pushed a commit that referenced this pull request Jan 14, 2022
docktermj added a commit that referenced this pull request Apr 4, 2022
docktermj added a commit that referenced this pull request Apr 5, 2022
docktermj added a commit that referenced this pull request Apr 5, 2022
* #1 Update version

* #1 Update version
docktermj added a commit that referenced this pull request May 9, 2022
docktermj added a commit that referenced this pull request May 9, 2022
docktermj added a commit that referenced this pull request May 16, 2022
docktermj added a commit that referenced this pull request May 16, 2022
docktermj added a commit that referenced this pull request May 17, 2022
docktermj added a commit that referenced this pull request May 17, 2022
docktermj added a commit that referenced this pull request May 31, 2022
docktermj added a commit that referenced this pull request Jan 12, 2023
* #1 Update version

* #1 Prepare for versioned release

* #1 Prepare for versioned release
docktermj added a commit that referenced this pull request Apr 3, 2023
docktermj added a commit that referenced this pull request May 9, 2023
docktermj added a commit that referenced this pull request May 9, 2023
docktermj added a commit that referenced this pull request Jun 15, 2023
docktermj added a commit that referenced this pull request Jun 15, 2023
* #1 Update version

* Prepare for versioned release
docktermj added a commit that referenced this pull request Jun 29, 2023
docktermj added a commit that referenced this pull request Jun 29, 2023
* #1 Update version

* Prepare for versioned release
docktermj added a commit that referenced this pull request Sep 29, 2023
docktermj added a commit that referenced this pull request Sep 29, 2023
* #1 Update version

* Update to template

* Update to template
docktermj added a commit that referenced this pull request Sep 29, 2023
@docktermj docktermj mentioned this pull request Sep 29, 2023
docktermj added a commit that referenced this pull request Sep 29, 2023
docktermj added a commit that referenced this pull request Nov 15, 2023
docktermj added a commit that referenced this pull request Nov 15, 2023
* #1 Update version

* Prepare for versioned release
kernelsam added a commit that referenced this pull request Mar 18, 2024
docktermj pushed a commit that referenced this pull request Mar 18, 2024
kernelsam added a commit that referenced this pull request May 22, 2024
kernelsam added a commit that referenced this pull request May 22, 2024
* #1 Update version

* prepare for versioned release
kernelsam added a commit that referenced this pull request Jun 24, 2024
kernelsam added a commit that referenced this pull request Jun 25, 2024
* #1 Update version

* Added USER root

* prepare for versioned release

---------

Co-authored-by: docktermj <[email protected]>
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