Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Poc grouping expenses #395

Merged
merged 11 commits into from
Nov 30, 2023
Merged

Poc grouping expenses #395

merged 11 commits into from
Nov 30, 2023

Conversation

ruuushhh
Copy link
Contributor

No description provided.

@ruuushhh ruuushhh requested a review from ashwin1111 November 24, 2023 09:03
@ruuushhh ruuushhh self-assigned this Nov 24, 2023
@github-actions GitHub Actions
Copy link

Tests Skipped Failures Errors Time
227 0 💤 0 ❌ 0 🔥 21.606s ⏱️

@github-actions GitHub Actions
Copy link

Tests Skipped Failures Errors Time
227 0 💤 0 ❌ 0 🔥 22.445s ⏱️

ashwin1111
ashwin1111 reviewed Nov 27, 2023
View reviewed changes
apps/fyle/models.py Outdated Show resolved Hide resolved
apps/fyle/models.py Outdated Show resolved Hide resolved
apps/fyle/models.py Outdated Show resolved Hide resolved
apps/fyle/models.py Outdated Show resolved Hide resolved
@github-actions GitHub Actions
Copy link

Tests Skipped Failures Errors Time
227 0 💤 0 ❌ 0 🔥 25.148s ⏱️

@codecov Codecov
Copy link

codecov bot commented Nov 27, 2023 •
edited
Loading

Codecov Report

Merging #395 (6660ea2) into master (5da783a) will increase coverage by 0.01%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #395      +/-   ##
==========================================
+ Coverage   97.29%   97.31%   +0.01%     
==========================================
  Files          53       53              
  Lines        4735     4759      +24     
==========================================
+ Hits         4607     4631      +24     
  Misses        128      128
Files Coverage Δ
apps/fyle/models.py 95.20% <100.00%> (+0.40%) ⬆️
apps/fyle/tasks.py 95.18% <100.00%> (+0.05%) ⬆️

ashwin1111
ashwin1111 reviewed Nov 27, 2023
View reviewed changes
apps/fyle/models.py Outdated Show resolved Hide resolved
apps/fyle/models.py Outdated Show resolved Hide resolved
apps/fyle/models.py Outdated Show resolved Hide resolved
@github-actions GitHub Actions
Copy link

Tests Skipped Failures Errors Time
227 0 💤 0 ❌ 0 🔥 24.616s ⏱️

@github-actions GitHub Actions
Copy link

Tests Skipped Failures Errors Time
228 0 💤 0 ❌ 0 🔥 25.843s ⏱️

@ruuushhh ruuushhh added the deploy Triggers deployment of active branch to Staging label Nov 29, 2023
@ruuushhh
Test cases added for grouping expenses (#397)
9f42c3d 
* Test cases added for grouping expenses

* Comment resolved
@github-actions GitHub Actions
Copy link

Tests Skipped Failures Errors Time
234 0 💤 0 ❌ 0 🔥 22.861s ⏱️

Shwetabhk
Shwetabhk approved these changes Nov 30, 2023
View reviewed changes
apps/fyle/models.py Outdated Show resolved Hide resolved
apps/fyle/models.py Outdated Show resolved Hide resolved
@github-actions GitHub Actions
Copy link

Tests Skipped Failures Errors Time
234 0 💤 0 ❌ 0 🔥 25.778s ⏱️

@ruuushhh ruuushhh merged commit a136a61 into master Nov 30, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Shwetabhk Shwetabhk Shwetabhk approved these changes

@ashwin1111 ashwin1111 ashwin1111 approved these changes

Assignees

@ruuushhh ruuushhh

Labels
deploy Triggers deployment of active branch to Staging
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ruuushhh @Shwetabhk @ashwin1111