Feature branch: Transactions and alerts #2037
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
* fix: swagger * fix: revert * fix: eslint --------- Co-authored-by: Lior Zamir <[email protected]>
* feat(*): the transactions drawer no longer uses hardcoded heading and summary * feat(backoffice-v2): added counterparty id to search query params * refactor(backoffice-v2): removed alert-definitions domain and moved logic to alerts domain * feat(backoffice-v2): added loading state to heading and summary * feat(backoffice-v2): added fallback state to alerts drawer * refactor(seed.ts): updated seed to alternative between business and counterparty id * revert(*): reverted counterparty id changes * fix(*): fixed changes that broke from conflicts * refactor(workflows-service): added optional chaining * fix(workflows-service): lint error
* fix(backoffice-v2): fixed filters in state but not in the url reaching the backend * refactor(backoffice-v2): removed console.log * revert(backoffice-v2): removed assignee default filter * fix(backoffice-v2): fixed key warning on navbar * fix(*): lockfile fix
* refactor(*): updated instances of alert state * refactor(backoffice-v2): updated toast text * refactor(backoffice-v2): simplified pattern matching for decision * refactor(workflows-service): addressed pr comment * fix(workflows-service): fixed return of function
github-actions
bot
force-pushed
the
alonp/feat/tm
branch
from
26e8c4e to
44ec84c
Compare
liorzam
force-pushed
the
alonp/feat/tm
branch
from
44ec84c to
676ec3b
Compare
github-actions
bot
force-pushed
the
alonp/feat/tm
branch
from
cffbbe9 to
32b9f17
Compare
liorzam
force-pushed
the
alonp/feat/tm
branch
from
32b9f17 to
676ec3b
Compare
liorzam
changed the title
…chema to support filtering by alert type fix(TransactionMonitoringAlertsAnalysis.page.tsx): add key prop to AlertAnalysisSheet component to force re-render when alertId changes fix(useTransactionMonitoringAlertsAnalysisPageLogic.tsx): remove duplicate navigateBack() call in useEffect hook fix(schema.prisma): fix indentation and rename underReview enum value to under_review for consistency fix(alert.service.ts): fix mapping of under_review enum value to pending status
github-actions
bot
force-pushed
the
alonp/feat/tm
branch
from
0812b7d to
6f00953
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Elaborate on the subject, motivation, and context.
Related issues
Breaking changes
How these changes were tested
Examples and references
Checklist