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

[UXE-6030] feat: update Real-Time Metrics, Events, Billing, and Accounting Services to API v4 with Origin Configuration Changes #2069

Open
wants to merge 8 commits into
base: dev
Choose a base branch
from

Conversation

HerbertJulio
Copy link
Collaborator

Pull Request

What is the new behavior introduced by this PR?

This pull request includes several updates to the codebase, primarily focused on upgrading the API version from v3 to v4 for various services and tests, along with changes to the origin configuration in azion.config.cjs.

Key Updates:

API Version Updates:

  • Updated the URL from v3/events/graphql to v4/events/graphql in:
    • cypress/e2mock/activity-history.cy.js
    • src/services/real-time-events-service/make-real-time-events-service.js
  • Updated the API version from v3 to v4 in:
    • src/services/real-time-metrics-services/make-beholder-base-url.js

Test Updates:

  • Updated the URL from v3/events/graphql to v4/events/graphql in the following test files:
    • src/tests/services/real-time-events-service/activity-history/list-activity-history.test.js
    • src/tests/services/real-time-events-service/activity-history/load-activity-history.test.js
    • src/tests/services/real-time-events-service/data-stream/list-data-stream.test.js
    • src/tests/services/real-time-events-service/data-stream/load-data-stream.test.js
    • And several other related test files.

Configuration Updates:

  • Changed the origin name from origin-manager to origin-api in azion.config.cjs [1] [2].

Does this PR introduce breaking changes?

  • Yes
    Services and tests dependent on v3 API endpoints will no longer work, as they have been updated to v4. Ensure all dependencies are aligned with the new API version.

Does this PR introduce UI changes? Add a video or screenshots here.

  • No

Does it have a link on Figma?

  • No

Checklist

Make sure your pull request fits the checklist below (when applicable):

  • The issue title follows the format: [ISSUE_CODE] TYPE: TITLE
  • Commits are tagged with the right word (feat, test, refactor, etc).
  • Application responsiveness was tested to different screen sizes (if applicable).
  • Code is formatted and linted.
  • Tags are added to the PR.

These changes were tested on the following browsers:

  • Chrome
  • Edge
  • Firefox
  • Safari

vinigfer
vinigfer previously approved these changes Jan 15, 2025
lucasmendes21
lucasmendes21 previously approved these changes Jan 16, 2025
pauloSF0
pauloSF0 previously approved these changes Jan 20, 2025
vinigfer
vinigfer previously approved these changes Jan 20, 2025
@HerbertJulio HerbertJulio dismissed stale reviews from vinigfer and pauloSF0 via 74d2e1a January 20, 2025 18:54
@HerbertJulio HerbertJulio force-pushed the UXE-6030-migracao-dos-servicos-gql-para-fora-do-manager-azion-com branch from 64720a0 to 635dd98 Compare January 20, 2025 19:30
pauloSF0
pauloSF0 previously approved these changes Jan 20, 2025
vinigfer
vinigfer previously approved these changes Jan 20, 2025
@HerbertJulio HerbertJulio dismissed stale reviews from vinigfer and pauloSF0 via 57ca170 January 27, 2025 22:09
Copy link

sonarqubecloud bot commented Feb 4, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
7.9% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

5 participants