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

Onboarding for Market Process Analyzer to AxonIvy Market #1

Conversation

ntqdinh-axonivy
Copy link
Collaborator

No description provided.

trungmaihova and others added 30 commits February 22, 2024 11:53
…test-find-all-task

TE-538: Update unit test with get longest path
…-example-processes

Added more example processes.
…ulate-estimated-time

TE-538: calculate estimated time
…-test-find-all-task

Te 538 unit test find all task
…-test-find-all-task

TE-538: Unit test findTasksOnPath with mixed
…e-example-processes

Feature/more example processes
…ulate-estimated-time

TE-538: modify logic and add unit test
…-test-find-all-task

TE-538: Fix to prevent loop
…-test-find-all-task

TE-538: Adapt Task type
…ulate-estimated-time

Te 538 calculate estimated time
…-test-find-all-task

TE-538: Sort by name for easy test
…-test-find-all-task

TE-538: Remove unuse code
…ulate-estimated-time

TE-538: add find task sub process
ntnchuong and others added 17 commits April 24, 2024 09:52
…ign-interfaces-for-DetectedElement-and-friends

TE-565: change product version
…actor-call-method

TE-567: Refactor call method
…actor-call-method

TE-567: Make function public
…dle-spent-time

TE-563: Next task is not impact by previous delay
…dle-complex-parallel-tasks

TE-571: handle complex parallel task
…dle-complex-parallel-tasks

TE-571: add default taskname for taskswitchgateway
…vert-to-task-group

TE-574: Convert to task parallel group
@ntqdinh-axonivy
Copy link
Collaborator Author

Hi @trungmaihova @ntnchuong
I have folked your dev branch to this PR before we can public market process analyzer to marketplace.
I also raise some feedback before we can release it, please check and do not hesitate to discuss with us if you feel some feedback can be ignore.
Btw, could you please confirm that the product name would be "Market Process Analyzer" or "Process Analyzer" ?
Thank you so much.

@trungmaihova
Copy link
Collaborator

Hi @trungmaihova @ntnchuong I have folked your dev branch to this PR before we can public market process analyzer to marketplace. I also raise some feedback before we can release it, please check and do not hesitate to discuss with us if you feel some feedback can be ignore. Btw, could you please confirm that the product name would be "Market Process Analyzer" or "Process Analyzer" ? Thank you so much.

@ivy-tru Please confirm the product name?

@ivy-tru
Copy link
Collaborator

ivy-tru commented May 13, 2024

Hi @ntqdinh-axonivy
Thanks for your review comments.

Btw, could you please confirm that the product name would be "Market Process Analyzer" or "Process Analyzer" ?

Official name of the market artifact should be "Advanced Process Analyzer". This name should already be used in the documentation files. It changed two times during the project, so sometimes we accidentally use the wrong name.

@ivy-rew
Copy link
Member

ivy-rew commented May 22, 2024

are we still waiting for anything @ivy-tru @ntnchuong @trungmaihova ?
I'd be happy to have a first version integrated asap.
As we have a build-pipeline that validates market-repos; and fails due to the 'virigin' state within the initial repo commit.
There's no need to enforce a publishing; just to merge this state to 'master' would help us already.

@ntqdinh-axonivy
Copy link
Collaborator Author

are we still waiting for anything @ivy-tru @ntnchuong @trungmaihova ?
I'd be happy to have a first version integrated asap.
As we have a build-pipeline that validates market-repos; and fails due to the 'virigin' state within the initial repo commit.
There's no need to enforce a publishing; just to merge this state to 'master' would help us already.

Hi @ivy-rew
There is still a small issue of the group id in the test module's POM file left before merging. We are in progress and will fix it soon tomorrow.
I'm sorry for our delay.

ntqdinh-axonivy pushed a commit that referenced this pull request May 23, 2024
…e-prototype

TE-538: Create WorkflowEstimator class
@ntqdinh-axonivy
Copy link
Collaborator Author

Hi @trungmaihova @ntnchuong
FYI @ivy-rew @ivy-rew @ivy-sgi
I would like to close this PR because the origin repo closed the folk connection and did not update the latest commit.
The follow-up PR is created #2 for onboarding the advanced process analyzer utils to market.

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.

5 participants