-
Notifications
You must be signed in to change notification settings - Fork 75
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(PaymentTask): Integrate Altinn Payment Task into ProcessEditor #12562
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #12562 +/- ##
==========================================
- Coverage 88.15% 88.13% -0.03%
==========================================
Files 1293 1293
Lines 18487 18494 +7
Branches 2265 2265
==========================================
+ Hits 16297 16299 +2
- Misses 1924 1929 +5
Partials 266 266 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Just remove the console.log :)
Unfortunately, I have no suggestions regarding testing the drag and drop with Playwright. Perhaps you want to keep this under review in case others have any suggestions.
…Altinn/altinn-studio into feat/addPaymentTaskToBpmnEditor
…12562) * feat(PaymentTask): added Altinn Payment Task to Editor
Description
I have made many attempts to drag and drop elements into the process editor using Playwright and can see that the element is being dropped correctly step by step, but unfortunately, the task itself is not being added to the diagram. When I test with actual drag and drop in the browser, it works as expected. I would appreciate any good suggestions if anyone has them. 😊
Related Issue(s)
Verification