FOUR-18762: Added PI Process Import to New Process modal#7319
FOUR-18762: Added PI Process Import to New Process modal#7319TinCodes merged 5 commits intoepic/FOUR-18761from
Conversation
| return this.$t('Import Process'); | ||
| }, | ||
| subtitle() { | ||
| return this.$t('Import a process from Workfellow PI (.jpg, .jpeg, .png) to this ProcessMaker environment'); |
There was a problem hiding this comment.
This text is not the same as the Figma. Should be:
Import a process from Workfellow PI to this ProcessMaker environment
There was a problem hiding this comment.
In sync with Claudia Iriarte and Franz Villaroel this text was to be included here to indicate the file type that the import accepts
There was a problem hiding this comment.
The file type should be JSON. Also need to update the figma
There was a problem hiding this comment.
Why we included into SelectTemplateModal? The name is confusing. Maybe we should have our own selectImportPIModal, or change the names to make it reusable
There was a problem hiding this comment.
This was to keep consistency with how the other 2 buttons (Create a new Process and Generate a Process with AI) work, since the TemplateSearch component emits these events to the SelectTemplateModal component. Let's sync to take a look at it!
| }, | ||
| subtitle() { | ||
| return this.$t('Import a process from Workfellow PI (.json) to this ProcessMaker environment'); | ||
| return this.$t('Import a process from Process Intelligence to this ProcessMaker environment'); |
There was a problem hiding this comment.
Please add the string "Import a process from Process Intelligence to this ProcessMaker environment" to the en.json file
Tickets Solved by this PR:
Solution
Code Review Checklist
ci:next