FOUR-18763: Implemented BE for PI Process import by TinCodes · Pull Request #7337 · ProcessMaker/processmaker
Navigation Menu
{{ message }}
ProcessMaker / processmaker Public
- Notifications You must be signed in to change notification settings
- Fork 246
Merged
FOUR-18763: Implemented BE for PI Process import#7337
TinCodes merged 2 commits intoepic/FOUR-18761from
FOUR-18763: Implemented BE for PI Process import#7337
TinCodes merged 2 commits intoepic/FOUR-18761from
Conversation
Copy link
Contributor
TinCodes
commented
Sep 6, 2024
TinCodes
commented
Tickets Solved by this PR:
Solution:
- Added new route on api.php for importing PI Processes
- Added new method on PackageAiController to send the pseudocode to the microservice
Related PRs:
Code Review Checklist
- I have pulled this code locally and tested it on my instance, along with any associated packages.
- This code adheres to ProcessMaker Coding Guidelines.
- This code includes a unit test or an E2E test that tests its functionality, or is covered by an existing test.
- This solution fixes the bug reported in the original ticket.
- This solution does not alter the expected output of a component in a way that would break existing Processes.
- This solution does not implement any breaking changes that would invalidate documentation or cause existing Processes to fail.
- This solution has been tested with enterprise packages that rely on its functionality and does not introduce bugs in those packages.
- This code does not duplicate functionality that already exists in the framework or in ProcessMaker.
- This ticket conforms to the PRD associated with this part of ProcessMaker.
ci:next
TinCodes
requested a review
from agustinbusso
agustinbusso requested changes Sep 6, 2024
| }, | ||
| importNewPI() {} | ||
| importNewPI() { | ||
| console.log('> ', this.file); |
Copy link
Contributor
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove this log
agustinbusso requested changes Sep 6, 2024
| }) | ||
| .catch((error) => { | ||
| window.ProcessMaker.alert( | ||
| this.$t("An error ocurred while importing the current PI Process."), |
Copy link
Contributor
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add to en.json file
TinCodes
requested a review
from agustinbusso
agustinbusso approved these changes Sep 6, 2024
agustinbusso approved these changes Sep 6, 2024
TinCodes
merged commit
06981b4
into
epic/FOUR-18761
This file contains hidden or 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants