Skip to content

Conversation

@aurelio-aot
Copy link
Contributor

No description provided.

@aurelio-aot aurelio-aot marked this pull request as draft January 23, 2026 04:24
@github-actions
Copy link

🧪 Unit Test Results (Parallel Execution)

Tests

📊 Summary

Result Count
✅ Passed 289
❌ Failed 0
⚠️ Skipped 0

📄 HTML Reports

  • Merged Tests (HTML): Included in artifacts
    Generated automatically by CI.

@github-actions
Copy link

🧪 Unit Test Results (Parallel Execution)

Tests

📊 Summary

Result Count
✅ Passed 289
❌ Failed 0
⚠️ Skipped 0

📄 HTML Reports

  • Merged Tests (HTML): Included in artifacts
    Generated automatically by CI.

@github-actions
Copy link

🧪 Unit Test Results (Parallel Execution)

Tests

📊 Summary

Result Count
✅ Passed 289
❌ Failed 0
⚠️ Skipped 0

📄 HTML Reports

  • Merged Tests (HTML): Included in artifacts
    Generated automatically by CI.

…as HTML

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@github-actions
Copy link

🧪 Unit Test Results (Parallel Execution)

Tests

📊 Summary

Result Count
✅ Passed 289
❌ Failed 0
⚠️ Skipped 0

📄 HTML Reports

  • Merged Tests (HTML): Included in artifacts
    Generated automatically by CI.

@aurelio-aot aurelio-aot marked this pull request as ready for review January 23, 2026 17:05
@JamesPasta
Copy link
Collaborator

This is Fantastic, it prevents Stephan's js from running though which unfortunately was used to show or hide sections.
I tried running this with some CGG forms and either we set the allowing of the evaluate to true and it runs all the js of the form or false and the show hide behavior on the page breaks.

There were about 5 sonar issues also but I dont' think we can merge this. Maybe we can merge just the css piece

@github-actions
Copy link

🧪 Unit Test Results (Parallel Execution)

Tests

📊 Summary

Result Count
✅ Passed 289
❌ Failed 0
⚠️ Skipped 0

📄 HTML Reports

  • Merged Tests (HTML): Included in artifacts
    Generated automatically by CI.

1 similar comment
@github-actions
Copy link

🧪 Unit Test Results (Parallel Execution)

Tests

📊 Summary

Result Count
✅ Passed 289
❌ Failed 0
⚠️ Skipped 0

📄 HTML Reports

  • Merged Tests (HTML): Included in artifacts
    Generated automatically by CI.

@aurelio-aot
Copy link
Contributor Author

This is Fantastic, it prevents Stephan's js from running though which unfortunately was used to show or hide sections. I tried running this with some CGG forms and either we set the allowing of the evaluate to true and it runs all the js of the form or false and the show hide behavior on the page breaks.

There were about 5 sonar issues also but I dont' think we can merge this. Maybe we can merge just the css piece

Hi @JamesPasta -- I have now removed the code that suppresses javascript inside the CHEFS form.io submission. I have also fixed the sonarqube issues.

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