Skip to content

feat(connector): create Taxjar connector#5597

Merged
likhinbopanna merged 4 commits intomainfrom
taxjar_connector_create
Aug 13, 2024
Merged

feat(connector): create Taxjar connector#5597
likhinbopanna merged 4 commits intomainfrom
taxjar_connector_create

Conversation

@ImSagnik007
Copy link
Contributor

@ImSagnik007 ImSagnik007 commented Aug 12, 2024

Type of Change

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

Description

This PR has template code for connector for TaxJar connector

How did you test it?

  • No test case Required

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
  • I added a CHANGELOG entry if applicable

@ImSagnik007 ImSagnik007 requested review from a team as code owners August 12, 2024 16:04
@pixincreate pixincreate changed the title feat(connector): craete Taxjar connector feat(connector): create Taxjar connector Aug 12, 2024
@ImSagnik007 ImSagnik007 force-pushed the taxjar_connector_create branch from 40d5259 to e32f97b Compare August 13, 2024 07:26
@swangi-kumari swangi-kumari added A-connector-integration Area: Connector integration C-feature Category: Feature request or enhancement labels Aug 13, 2024
Copy link
Contributor

@deepanshu-iiitu deepanshu-iiitu left a comment

Choose a reason for hiding this comment

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

LGTM

@likhinbopanna likhinbopanna added this pull request to the merge queue Aug 13, 2024
Merged via the queue into main with commit 0ab0aa1 Aug 13, 2024
@likhinbopanna likhinbopanna deleted the taxjar_connector_create branch August 13, 2024 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-connector-integration Area: Connector integration C-feature Category: Feature request or enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants