-
Notifications
You must be signed in to change notification settings - Fork 8
feat/AB#105869_EIS-allow-data-field-calculation-for-fields-coming-from-resource-as-well #2699
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
base: beta
Are you sure you want to change the base?
Conversation
…m-resource-as-well feat: make sure that content inside span tags can be correctly formatted and sent back using the date calc feature of the html parser
it seems it's working in most cases, but I found some cases where it doesn't You can test them in the DEV environment, in the Antoine - EIS application Events page, event groups tab returns: the first one seems to work, but not the second one. Most cards of this view have a similar issue, only one date working fine Event details POC It works fine if I untick the "use layout styles" in the settings: But it doesn't work if I let the option: ( please don't put any screenshot from these data, or, as I did, crop it so we only see it partially ) |
ey @AntoineRelief , is there something that I might missing on the configuration for the cases in the events group tab? About the second point, this is the regex we are using The point is that those content with style(e.g. background-color) have inner My question is, is there a reason on why we dont want to capture any closing You can do test with this html
|
if you're taking it from the app I shared, you shouldn't miss anything I believe about the second point, that's a quite good question and I have absolutely no idea |
…m-resource-as-well feat: add option to execute nested calc operations for the given html content
@AntoineRelief extracted from the page examples you gave me |
…m-resource-as-well fix: infinite loop
…ion-for-fields-coming-from-resource-as-well # Please enter a commit message to explain why this merge is necessary, # especially if it merges an updated upstream into a topic branch. # # Lines starting with '#' will be ignored, and an empty message aborts # the commit.
did I miss something? 😅 It doesn't work for me in all cases for summary card |
Can you give me the exact application and page? |
I
I wanted to check something about that you say:
but CALC_SUFFIX doesn't exist, and if I try to replace with PLACEHOLDER_SUFFIX, I get thousands of errors: are you sure about this one you tested? |
Yes, if not try the testing through the application, it worked, the previous images are taken for the last version of the branch within the applicattion. |
Description
feat: make sure that content inside span tags can be correctly formatted and sent back using the date calc feature of the html parser
Useful links
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.
Screenshots
Compartir.pantalla.-.2024-11-18.10_06_16.1.mp4
Checklist:
( * == Mandatory )
More explanation
https://www.loom.com/share/05a716d61b9744faaf51fb304c21d1e5?sid=f87cf896-582a-4f76-93ae-8ceed801b145