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

kie-issues#2548: Cannot add a new record within an empty Decision Table #2912

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Kusuma04-dev
Copy link
Contributor

closes #2548 Cannot add a new record within an empty decision table

Before fix:
When user added empty decision table without any row , + icon is not visible to add rows.

After fix:
If user added an empty decision table then row is displayed, if user added something in that row then only it is updated otherwise row is just for display purpose and here if user tries to add another row , he can able to add after updating something in the displayed row.

@jomarko jomarko self-requested a review February 19, 2025 08:32
Copy link
Contributor

@jomarko jomarko left a comment

Choose a reason for hiding this comment

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

@Kusuma04-dev Hi, thank you for the PR. I think from now one users will be able to add rows even to a table, that has zero rows/rules in the xml.

However, I would like to share one idea/proposal, that from my point of view would make more clear difference between no data state vs some data state.

Currently we have

0 rows just opened

Screenshot 2025-02-19 095435

0 rows transformed to one row

Screenshot 2025-02-19 095445

Proposal

0 rows just opened

Screenshot 2025-02-19 095500

0 rows transformed to one row

Screenshot 2025-02-19 095435

@Kusuma04-dev
Copy link
Contributor Author

@Kusuma04-dev Hi, thank you for the PR. I think from now one users will be able to add rows even to a table, that has zero rows/rules in the xml.

However, I would like to share one idea/proposal, that from my point of view would make more clear difference between no data state vs some data state.

Currently we have

0 rows just opened

Screenshot 2025-02-19 095435

0 rows transformed to one row

Screenshot 2025-02-19 095445

Proposal

0 rows just opened

Screenshot 2025-02-19 095500

0 rows transformed to one row

Screenshot 2025-02-19 095435

Thanks Jozef for the review and the proposal. I agree with you ,with these changes user can distinguish between dummy row and actual row.

@Kusuma04-dev
Copy link
Contributor Author

Kusuma04-dev commented Feb 19, 2025

@tiagobento , Can you please share your thoughts on this:
Can we do like this as jozef mentioned:

Screen.Recording.2025-02-19.at.6.16.31.PM.mov

@tiagobento
Copy link
Contributor

IDK I think it's weird to have to click on the + sign to add a row that is already there. If we always show at lest one row (even if the DMN XML says the Decision Table has 0 rows), users will be able to use normally...

@tiagobento tiagobento changed the title kie-issues#2548: Cannot add a new record within an empty decision table kie-issues#2548: Cannot add a new record within an empty Decision Table Feb 19, 2025
@Kusuma04-dev
Copy link
Contributor Author

IDK I think it's weird to have to click on the + sign to add a row that is already there. If we always show at lest one row (even if the DMN XML says the Decision Table has 0 rows), users will be able to use normally...

@tiagobento ,Thanks for your suggestion , Yeah user can use normally ,here 1st row which is displayed is not actual rule , it becomes active only when user added something then they can use normally but if user didn't add anything then clicked on + icon then it will add 1st row(considering no rule present before). Please share your view on this if this is fine or we need to modify anything?

@Kusuma04-dev
Copy link
Contributor Author

Hi @tiagobento @jomarko , I have made the changes to make it work as normal decisiontable, Please review and suggest if anything is missing.

@Kusuma04-dev Kusuma04-dev requested a review from jomarko February 20, 2025 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot add a new record within an empty decision table
3 participants