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

[DSIP-68] Uniformly name process in code as workflow #16515

Closed
2 tasks done
Tracked by #14102
SbloodyS opened this issue Aug 27, 2024 · 2 comments · Fixed by #16544
Closed
2 tasks done
Tracked by #14102

[DSIP-68] Uniformly name process in code as workflow #16515

SbloodyS opened this issue Aug 27, 2024 · 2 comments · Fixed by #16544
Assignees

Comments

@SbloodyS
Copy link
Member

SbloodyS commented Aug 27, 2024

Search before asking

  • I had searched in the DSIP and found no similar DSIP.

Motivation

Uniformly name process in code as workflow for better maintenance

  1. First modify the backend code uniformly [Improvement] Uniformly name process in backend code as workflow #16516
  2. Then unify the table structure and table fields in backend [Improvement] Unify the table structure and table fields in backend and frontend ui sections #16525
  3. Unify the backend controller and frontend ui sections [Improvement] Unify the table structure and table fields in backend and frontend ui sections #16525

Design Detail

Incompatible changes

  • Rename tables and fields from process to workflow. eg, from t_ds_process_definition to t_ds_workflow_definition.
  • Rename api request/response params from process to workflow. eg, from processDefinitionCode to WorkflowDefinitionCode.

These incompatible changes may affect users of secondary development based on API and Database.

Compatibility, Deprecation, and Migration Plan

No response

Test Plan

Test it with ut, it, e2e, api-test

Code of Conduct

@SbloodyS SbloodyS added the DSIP label Aug 27, 2024
@SbloodyS SbloodyS self-assigned this Aug 27, 2024
@SbloodyS SbloodyS changed the title [DSIP-68] Uniformly name process in your code as workflow [DSIP-68] Uniformly name process in code as workflow Aug 27, 2024
@SbloodyS SbloodyS mentioned this issue Aug 27, 2024
74 tasks
@SbloodyS SbloodyS added the discussion discussion label Aug 27, 2024
@zhongjiajie
Copy link
Member

Maybe we should find out whether another column name should change, related to this issue to avoid too many incompatible upgrades. checking ourselves or collecting from the community

@SbloodyS
Copy link
Member Author

Maybe we should find out whether another column name should change, related to this issue to avoid too many incompatible upgrades. checking ourselves or collecting from the community

Only the upgrade version will be affected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants