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

feature/bull_mq_integration #2735

Open
wants to merge 18 commits into
base: dev
Choose a base branch
from
Open

feature/bull_mq_integration #2735

Show file tree
Hide file tree
Changes from 16 commits
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
4632b91
feat: added redis defaults
Blokh Sep 29, 2024
faaacf5
added bull mq configuration logic
Blokh Sep 29, 2024
44343e8
added bull mq configuration logic
Blokh Sep 29, 2024
8e1ff70
updated bull functionality
Blokh Sep 30, 2024
00ec074
intial webhook queue logic
Blokh Sep 30, 2024
146d628
added outgoing and incoming queues logic
Blokh Sep 30, 2024
5b83c6e
feat: split outoing and incoming hook functionality and created queue…
Blokh Oct 7, 2024
606a006
updated worker logic to use different appmodule
Blokh Oct 14, 2024
63e6172
Merge branch 'dev' into feature/bull_mq_integration
Blokh Oct 14, 2024
e7d416b
feat: updated logic of webhook functionality
Blokh Oct 14, 2024
ab614af
generated process of adding queue for each event emitter
Blokh Oct 14, 2024
dbf0374
updated workflow module
Blokh Oct 14, 2024
a26bb97
Merge branch 'dev' into feature/bull_mq_integration
Blokh Oct 20, 2024
216bd60
updated env logic
Blokh Oct 20, 2024
ef46137
feat: fixed tests
Blokh Oct 20, 2024
b20a326
Feature/implement bull dlq (#2792)
Blokh Oct 22, 2024
3193904
removed unnecessary thro werror
Blokh Oct 22, 2024
db1999f
fix(slack): fix mm customer name
alonp99 Nov 22, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions apps/backoffice-v2/src/lib/blocks/components/EditableDetails/EditableDetails.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,7 @@ export const Detail: FunctionComponent<IDetailProps> = ({

return value;
};

const value = getValue(children);

return (
Expand Down
1 change: 1 addition & 0 deletions apps/backoffice-v2/src/lib/blocks/hooks/useDocumentBlocks/useDocumentBlocks.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -409,6 +409,7 @@ export const useDocumentBlocks = ({
}

if (ocrResult?.parsedData?.[title]) {
debugger;
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue

Remove the debugger statement to prevent unintended pauses in execution.

The debugger; statement on line 412 can cause the application to halt unexpectedly, which is undesirable in production code.

Apply this diff to remove the debugger statement:

-                              debugger;
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
debugger;
🧰 Tools
🪛 Biome

[error] 412-412: This is an unexpected use of the debugger statement.

Unsafe fix: Remove debugger statement

(lint/suspicious/noDebugger)

const isOcrValueString = typeof ocrResult.parsedData[title] === 'string';

if (isOcrValueString && ocrResult.parsedData[title].length > 0) {
Expand Down
Loading
Loading