Skip to content

Remove nym-network-statistics #4678

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

Merged
merged 4 commits into from
Jun 27, 2024
Merged

Conversation

octol
Copy link
Contributor

@octol octol commented Jun 26, 2024

Remove nym-network-statistics service provider that is no longer used

CT-185

Copy link

vercel bot commented Jun 26, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Ignored Deployments
Name Status Preview Comments Updated (UTC)
nym-explorer ⬜️ Ignored (Inspect) Visit Preview Jun 26, 2024 1:26pm
nym-next-explorer ⬜️ Ignored (Inspect) Visit Preview Jun 26, 2024 1:26pm

@octol octol marked this pull request as ready for review June 26, 2024 13:18
@octol octol added this to the Wispa milestone Jun 26, 2024
@octol octol requested a review from jstuczyn June 26, 2024 13:21
@octol octol merged commit 14a904e into develop Jun 27, 2024
13 checks passed
@octol octol deleted the jon/remove-nym-network-statistics branch June 27, 2024 08:32
@tommyv1987
Copy link
Contributor

Testing Completion Date: 23/07/2024

Testing Steps Performed:

  1. Reviewed the project to identify all references to nym-network-statistics.
  2. Removed all code and dependencies associated with nym-network-statistics.
  3. Ensured that no references to nym-network-statistics remain in the codebase, including comments, imports, and configuration files.
  4. Compiled the project to check for any issues or warnings.
  5. Ran all tests to ensure the removal did not affect the functionality of the project.

Validation:

  • Test Passed
  • Issues Found (Please specify below)

Notes (if any):
The test was successful. All references to nym-network-statistics were removed, and the project compiled correctly. No issues were found during testing.

Attachments (if applicable):
No attachments provided.

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