Skip to content

update data path#835

Merged
chakravarthik27 merged 1 commit intorelease/1.7.0from
feature/finance-test
Oct 19, 2023
Merged

update data path#835
chakravarthik27 merged 1 commit intorelease/1.7.0from
feature/finance-test

Conversation

@chakravarthik27
Copy link
Copy Markdown
Collaborator

@chakravarthik27 chakravarthik27 commented Oct 19, 2023

Description

➤ Fixes # (issue)

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Usage

Checklist:

  • I've added Google style docstrings to my code.
  • I've used pydantic for typing when/where necessary.
  • I have linted my code
  • I have added tests to cover my changes.

Screenshots (if appropriate):

@chakravarthik27 chakravarthik27 added the 💡Enhancements Something can be improved label Oct 19, 2023
@chakravarthik27 chakravarthik27 self-assigned this Oct 19, 2023
@chakravarthik27 chakravarthik27 merged commit f21cf5f into release/1.7.0 Oct 19, 2023
@ArshaanNazir ArshaanNazir deleted the feature/finance-test branch November 1, 2023 06:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@Prikshit7766 Prikshit7766 Prikshit7766 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@chakravarthik27 chakravarthik27

Labels

💡Enhancements Something can be improved

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@chakravarthik27 @Prikshit7766