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.

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

feat: Filter transactions table by date #993

Open
wants to merge 57 commits into
base: develop
Choose a base branch
from

Conversation

janmichek
Copy link
Collaborator

@janmichek janmichek commented Nov 14, 2024

Description

fixes #615

Demo

Checklist:

  • I have read and followed the Contributing Guide
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@janmichek janmichek changed the title fet: Filter transactions table by date feat: Filter transactions table by date Nov 14, 2024
@janmichek janmichek force-pushed the Filter-transactions-table-by-date branch from 2cfef85 to 2dfaa39 Compare November 15, 2024 13:32
@@ -28,4 +28,61 @@ describe('transactions', () => {
})
})
})

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This was really hard to achive, hence is fully tested

@@ -8,16 +8,16 @@
</template>
<template #end>
<chart-controls
v-model="selectedRange"
v-model="selectedScope"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

renaming of sibling component needed to be refactored

pageLimit.value = value
}

function formatSlug(scope, type) {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

there is a lot of formatting function. After several attempts this is the minimal version

@janmichek
Copy link
Collaborator Author

@michele-franchi In the meantime while waiting for minor MDW fix you can do the review. I will leave it On Hold column for automation purposes.
Only thing that is not working for now is the counter of transactions when combining parameters.

@janmichek janmichek added this to the Q4 2024 milestone Jan 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Filter transactions table by date
1 participant