Skip to content

Conversation

shivamgupta88
Copy link

@shivamgupta88 shivamgupta88 commented Aug 30, 2025

…s cloning

Fixed invalid path 'postman/collection-dir/.../Scenario10-Create 3DS payment without external 3ds authentication /.meta.json' by removing trailing space from directory name. This resolves the Windows filesystem compatibility issue where trailing spaces in file/folder names are not allowed, causing git clone to fail on Windows.

Fixes #9111

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

…s cloning

Fixed invalid path 'postman/collection-dir/.../Scenario10-Create 3DS payment without external 3ds authentication /.meta.json'
by removing trailing space from directory name. This resolves the Windows filesystem compatibility issue
where trailing spaces in file/folder names are not allowed, causing git clone to fail on Windows.

Fixes juspay#9111

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@shivamgupta88 shivamgupta88 requested a review from a team as a code owner August 30, 2025 12:19
Copy link

semanticdiff-com bot commented Aug 30, 2025

Review changes with  SemanticDiff

Changed Files
File Status
  crates/hyperswitch_connectors/src/connectors/nuvei/transformers.rs  0% smaller
  crates/router/src/core/api_locking.rs  0% smaller
  crates/router/src/core/payment_methods.rs  0% smaller
  crates/router/src/core/unified_connector_service.rs  0% smaller
  postman/collection-dir/archipel/Flow Testcases/Happy Cases/Scenario10-Create 3DS payment without external 3ds authentication/.meta.json  0% smaller
  postman/collection-dir/archipel/Flow Testcases/Happy Cases/Scenario10-Create 3DS payment without external 3ds authentication/Payments - Confirm/.event.meta.json  0% smaller
  postman/collection-dir/archipel/Flow Testcases/Happy Cases/Scenario10-Create 3DS payment without external 3ds authentication/Payments - Confirm/event.prerequest.js  0% smaller
  postman/collection-dir/archipel/Flow Testcases/Happy Cases/Scenario10-Create 3DS payment without external 3ds authentication/Payments - Confirm/event.test.js  0% smaller
  postman/collection-dir/archipel/Flow Testcases/Happy Cases/Scenario10-Create 3DS payment without external 3ds authentication/Payments - Confirm/request.json  0% smaller
  postman/collection-dir/archipel/Flow Testcases/Happy Cases/Scenario10-Create 3DS payment without external 3ds authentication/Payments - Confirm/response.json  0% smaller
  postman/collection-dir/archipel/Flow Testcases/Happy Cases/Scenario10-Create 3DS payment without external 3ds authentication/Payments - Create/.event.meta.json  0% smaller
  postman/collection-dir/archipel/Flow Testcases/Happy Cases/Scenario10-Create 3DS payment without external 3ds authentication/Payments - Create/event.prerequest.js  0% smaller
  postman/collection-dir/archipel/Flow Testcases/Happy Cases/Scenario10-Create 3DS payment without external 3ds authentication/Payments - Create/event.test.js  0% smaller
  postman/collection-dir/archipel/Flow Testcases/Happy Cases/Scenario10-Create 3DS payment without external 3ds authentication/Payments - Create/request.json  0% smaller
  postman/collection-dir/archipel/Flow Testcases/Happy Cases/Scenario10-Create 3DS payment without external 3ds authentication/Payments - Create/response.json  0% smaller
  postman/collection-dir/archipel/Flow Testcases/Happy Cases/Scenario10-Create 3DS payment without external 3ds authentication/Payments - Retrieve after confirm/.event.meta.json  0% smaller
  postman/collection-dir/archipel/Flow Testcases/Happy Cases/Scenario10-Create 3DS payment without external 3ds authentication/Payments - Retrieve after confirm/event.test.js  0% smaller
  postman/collection-dir/archipel/Flow Testcases/Happy Cases/Scenario10-Create 3DS payment without external 3ds authentication/Payments - Retrieve after confirm/request.json  0% smaller
  postman/collection-dir/archipel/Flow Testcases/Happy Cases/Scenario10-Create 3DS payment without external 3ds authentication/Payments - Retrieve after confirm/response.json  0% smaller

pixincreate
pixincreate previously approved these changes Aug 31, 2025
@pixincreate pixincreate added A-CI-CD Area: Continuous Integration/Deployment S-waiting-on-review Status: This PR has been implemented and needs to be reviewed labels Aug 31, 2025
shivamgupta88 and others added 2 commits September 1, 2025 16:56
- Fix 'aqcuired' -> 'acquired' in api_locking.rs
- Fix 'addtional' -> 'additional' in payment_methods.rs
- Fix 'fo' -> 'for' in nuvei transformers.rs

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@@ -1404,7 +1404,7 @@ where
card_holder_name: data.card_holder_name,
expiration_month: Some(data.card_exp_month),
expiration_year: Some(data.card_exp_year),
..Default::default() // CVV should be disabled by nuvei fo
..Default::default() // CVV should be disabled by nuvei for
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
..Default::default() // CVV should be disabled by nuvei for
..Default::default() // CVV should be disabled for nuvei

correct me if wrong, @Nithin1506200

Copy link
Member

Choose a reason for hiding this comment

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

also, you do not need to merge from main when this branch goes out of date with the base branch (main).

pixincreate
pixincreate previously approved these changes Sep 2, 2025
Copy link
Member

@pixincreate pixincreate left a comment

Choose a reason for hiding this comment

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

looks good, thanks for the contribution, @shivamgupta88💐!

likhinbopanna
likhinbopanna previously approved these changes Sep 2, 2025
@pixincreate pixincreate changed the title fix(postman): Remove trailing space from directory path to fix Window… chore: remove trailing space from directory paths in postman and address typos in the repo Sep 3, 2025
@pixincreate
Copy link
Member

@shivamgupta88, can you please update the PR description by moving it to the right place? also update the description to include the typos being fixed in the repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-CI-CD Area: Continuous Integration/Deployment S-waiting-on-review Status: This PR has been implemented and needs to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Repository fails to clone on Windows due to invalid file path in postman/collection-dir/.../.meta.json
3 participants