Skip to content

New 2#2

Open
itsharshvb wants to merge 8 commits intomainfrom
new-2
Open

New 2#2
itsharshvb wants to merge 8 commits intomainfrom
new-2

Conversation

@itsharshvb
Copy link
Copy Markdown
Owner

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

Additional Changes

  • This PR modifies the API contract
  • This PR modifies the database schema
  • This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

Checklist

  • I formatted the code cargo +nightly fmt --all
  • I addressed lints thrown by cargo clippy
  • I reviewed the submitted code
  • I added unit tests for my changes where possible

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Sep 26, 2025

API Compatibility Report

Status: ALL CHECKS PASSED

Summary

Metric V1 API V2 API Total
Breaking Changes 2 0 2
Spectral Errors 0 0 0
Spectral Warnings 0 0 0
New Endpoints 0 0 0
Removed Endpoints 0 0 0
Modified Endpoints 0 0 0

Status: BREAKING CHANGES DETECTED

This change introduces breaking changes that will affect existing API clients.


Breaking Changes Detected

V1 API Breaking Changes (2 issues)

  • 1 changes: 1 error, 0 warning, 0 info
  • error [api-path-removed-without-deprecation] at base-v1-schema.json
  • in API GET /account/payment_methods
  •   api path removed without deprecation
    

... and 1 more issues


Recommendations

Breaking changes detected - Review carefully before merging:

  • Consider API versioning instead of modifying existing endpoints
  • Coordinate with API consumers before deployment
  • Update documentation with breaking change notices

Generated at: Fri Sep 26 04:06:13 UTC 2025


Local Testing

Test these changes locally:

# Run full validation
just api-validate

# Compare with specific branch/commit
just api-diff origin/main HEAD

Automatically generated by the API Compatibility workflow.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant