canonical / ubuntu.com

The official website for the Ubuntu operating system
https://ubuntu.com
Other
201 stars 199 forks source link

Fix containers forms and remove unused forms #14183

Closed britneywwc closed 2 months ago

britneywwc commented 2 months ago

Done

QA

Issue / Card

Fixes WD-14054

Screenshots

[If relevant, please include a screenshot.]

Help

QA steps - Commit guidelines

webteam-app commented 2 months ago

Demo

Jenkins

demos.haus

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 69.57%. Comparing base (753131a) to head (8381d1e). Report is 21 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #14183 +/- ## ======================================= Coverage 69.57% 69.57% ======================================= Files 120 120 Lines 3412 3412 Branches 1176 1176 ======================================= Hits 2374 2374 Misses 1013 1013 Partials 25 25 ```
akbarkz commented 2 months ago

@britneywwc I noticed there is an error in dynamic-forms.js file when you close a containers form modal. Here's the screenshot of the error. Can you please find out why? Screenshot 2024-08-19 at 12 38 12