Skip to content

Added the label change for spiff#82

Merged
auslin-aot merged 3 commits intoAOT-Technologies:mainfrom
abilpraju-aot:bugfix/M8F-150-Label-change
Mar 19, 2026
Merged

Added the label change for spiff#82
auslin-aot merged 3 commits intoAOT-Technologies:mainfrom
abilpraju-aot:bugfix/M8F-150-Label-change

Conversation

@abilpraju-aot
Copy link
Collaborator

@abilpraju-aot abilpraju-aot commented Mar 18, 2026

JIRA Ticket

https://aottech.atlassian.net/browse/M8F-150
image

Description

Overrided the label for spiff service property

Type

  • Feature
  • Bug fix
  • Documentation
  • Other

Changes

  • Backend
  • Frontend
  • Documentation

Testing

Related Issues

Closes #

@abilpraju-aot abilpraju-aot marked this pull request as ready for review March 18, 2026 05:13
const replacements = [
{
from: "translate('Spiffworkflow Service Properties')",
to: "translate('m8flow Service Properties')",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

M8Flow should have the "M" capitalized.


for (const { from, to } of replacements) {
if (content.includes(from)) {
content = content.replace(from, to);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is just a work around. If it's just treated as so, it's ok, but we need to discuss it and consider forking sartography/bpmn-js-spiffworkflow as we may need to do other customizations.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ya agree with your point this is only for quick fix

@sonarqubecloud
Copy link

@auslin-aot auslin-aot merged commit e894d55 into AOT-Technologies:main Mar 19, 2026
11 of 12 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