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

fix(frontend): select fields display value textmodule inconsistency #4917

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

arsa-dev
Copy link

@arsa-dev arsa-dev commented Nov 3, 2023

Related to #2745

This is an implementation that tries to follow notification factory renderer implementation merged with what I have seen for getting attributes on frontend.

Steps to reproduce

20231103_23-05-30_AR_Zammad Helpdesk - #27002 - test

20231103_23-05-09_AR_Zammad Helpdesk - #27002 - test

20231103_23-04-52_AR_Zammad Helpdesk - #27002 - test

Tests made after changes

20231104_00-19-20_AR_Zammad Helpdesk - #27002 - test

20231104_00-19-48_AR_Zammad Helpdesk - #27002 - test

@dominikklein
Copy link
Collaborator

@arsa-dev Thanks for your contribution, hopefully, we can check this changes in the next weeks. Maybe you have the time to resolve the current conflicts?

@arsa-dev arsa-dev force-pushed the fix-select-fields-textmodule-inconsistency branch from 3b7ef82 to 85ad2d7 Compare February 26, 2024 20:23
Signed-off-by: Antonio Ramón Sánchez Morales <antonioramonsm@gmail.com>
@arsa-dev arsa-dev force-pushed the fix-select-fields-textmodule-inconsistency branch from 85ad2d7 to 5b78828 Compare February 26, 2024 20:40
@arsa-dev
Copy link
Author

Hi @dominikklein, sorry for the delay. Did not had much time since your last comment but finally I had a bit of time today to rebase my changes and resolve conflicts with 9fd9e34#diff-ca9eabd9c8e72ad4811a516842cec350f4a62a6637b7fc5443ac860dd46e742f.

I expect to be more available on next days so I'll probably will have some time to follow faster if there is any other indication or change to perform.

Thanks and regards, Antonio

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.

None yet

2 participants