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

Developer App Analytics page for Hybrid Org #105 #123

Merged
merged 14 commits into from
Mar 2, 2021

Conversation

phdhiren
Copy link
Contributor

@phdhiren phdhiren commented Feb 1, 2021

Analytics page for Hybrid org, there was change in API end points for hybrid orgs

Reference issues:

#105
apigee/apigee-edge-drupal#552
apigee/apigee-edge-drupal#481

Add support to hybrid organization for analytics funtions
Add support to hybrid organization for getMetrics & cs fix
@google-cla google-cla bot added the cla: yes Indicates CLA has been signed label Feb 1, 2021
fix phpdoc_tag_type
query_params to be initialize
@phdhiren phdhiren marked this pull request as ready for review February 24, 2021 07:25
@phdhiren phdhiren linked an issue Feb 25, 2021 that may be closed by this pull request
Copy link
Contributor

@raakesh-blokhra raakesh-blokhra left a comment

Choose a reason for hiding this comment

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

LGTM thanks

@phdhiren
Copy link
Contributor Author

The pull request will fix apigee/apigee-edge-drupal#481

raakesh-blokhra added a commit that referenced this pull request Apr 19, 2021
* restrict polyfill-mbstring to 1.20.0 / Fix travis-ci (#121)

Added support for PHP 7.3 + Fix .travis.yml  + restrict polyfill-mbstring to 1.20.0

* Developer App Analytics page for Hybrid Org #105 (#123)

Resolved Error on Developer App Analytics page for Hybrid Org #105

* Closes-#108 - After creating app without description throwing php not… (#125)

* Closes-#108 - After creating app without description throwing php notice in hybrid - fixed

* App Analytics page - Fix PHP error on empty data  (#133)

Fix PHP error in case of empty data for Analytics page

Co-authored-by: phdhiren <71828912+phdhiren@users.noreply.github.com>
Co-authored-by: Shishir <75600200+shishir-intelli@users.noreply.github.com>
raakesh-blokhra added a commit that referenced this pull request Apr 19, 2021
* restrict polyfill-mbstring to 1.20.0 / Fix travis-ci in feature-ngmint (#132)

* restrict polyfill-mbstring to 1.20.0
* Enabled PHP 7.1, 7.2, 7.3
* FIx travis-ci for feature-ngmint

* Code for Purchase,Cancel,Listing of Purchased Products (#131)

* Code for Purchase,Cancel,Listing of Purchased Products

* phpcs fix

* phpcsfixer

* code cleanup phpcs

* phpcsfix

* phpcs fixes

* phpcs fix

* phpcsfixes

Co-authored-by: Hirenkumar Parmar <hirenkumar@google.com>
Co-authored-by: phdhiren <71828912+phdhiren@users.noreply.github.com>

* Initial Commit for NGMint (ApigeeX 5G) (#130)

* Initial Commit for NGMint

* Feature ngmint - updated by pulling code from 2.x (#136)

* restrict polyfill-mbstring to 1.20.0 / Fix travis-ci (#121)

Added support for PHP 7.3 + Fix .travis.yml  + restrict polyfill-mbstring to 1.20.0

* Developer App Analytics page for Hybrid Org #105 (#123)

Resolved Error on Developer App Analytics page for Hybrid Org #105

* Closes-#108 - After creating app without description throwing php not… (#125)

* Closes-#108 - After creating app without description throwing php notice in hybrid - fixed

* App Analytics page - Fix PHP error on empty data  (#133)

Fix PHP error in case of empty data for Analytics page

Co-authored-by: phdhiren <71828912+phdhiren@users.noreply.github.com>
Co-authored-by: Shishir <75600200+shishir-intelli@users.noreply.github.com>

Co-authored-by: phdhiren <71828912+phdhiren@users.noreply.github.com>
Co-authored-by: divya-intelli <75604843+divya-intelli@users.noreply.github.com>
Co-authored-by: Hirenkumar Parmar <hirenkumar@google.com>
Co-authored-by: Shishir <75600200+shishir-intelli@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Indicates CLA has been signed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error on Developer App Analytics page for Hybrid Org
4 participants