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

"Added generatePipelineId function to blockUtils and updated Drawflow… #195

Merged
merged 5 commits into from
Aug 29, 2024

Conversation

TeerthaDeb
Copy link
Contributor

…Wrapper to use it to generate a new ID for pipelines"

Copy link

sentry-io bot commented Aug 15, 2024

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: frontend/src/components/ui/DrawflowWrapper.jsx

Function Unhandled Issue
dropPipeline TypeError: Cannot read properties of undefined (reading 'information') generateId...
Event Count: 1 Affected Users: 1
dropPipeline ReferenceError: generatePipelineId is not defined dropPipeline(src/components/ui/DrawflowWrapper...
Event Count: 1 Affected Users: 1
📄 File: frontend/src/utils/blockUtils.js (Click to Expand)
Function Unhandled Issue
generateId TypeError: Cannot read properties of undefined (reading 'information') generateId...
Event Count: 1 Affected Users: 1
---

Did you find this useful? React with a 👍 or 👎

frontend/src/components/ui/DrawflowWrapper.jsx Outdated Show resolved Hide resolved
frontend/src/utils/blockUtils.js Outdated Show resolved Hide resolved
frontend/src/utils/blockUtils.js Outdated Show resolved Hide resolved
frontend/src/utils/blockUtils.js Outdated Show resolved Hide resolved
frontend/src/atoms/pipelineAtom.js Outdated Show resolved Hide resolved
frontend/src/components/ui/DrawflowWrapper.jsx Outdated Show resolved Hide resolved
frontend/src/utils/blockUtils.js Outdated Show resolved Hide resolved
@TeerthaDeb TeerthaDeb requested a review from jmagoon August 21, 2024 15:24
@TeerthaDeb TeerthaDeb force-pushed the i-branch branch 2 times, most recently from 313ebaf to 1e9d3c7 Compare August 23, 2024 15:47
@jmagoon
Copy link
Member

jmagoon commented Aug 23, 2024

This MR looks good, I'm merging it into f/deploy for testing how it affects pipeline history and will merge it with that

@jmagoon jmagoon merged commit 99806c2 into zetane:main Aug 29, 2024
8 checks passed
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.

3 participants