custom function edge delivery integration

[deepprakash345]

No description provided.

[adobe-bot]

Lighthouse scores (mobile)

	Performance	Accessibility	Best-Practices	SEO
Scores	98	96	96	75

[adobe-bot]

Lighthouse scores (desktop)

	Performance	Accessibility	Best-Practices	SEO
Scores	100	96	96	75

[adobe-bot]

Accessibility Violations Found

Id	Impact
label-title-only	serious
landmark-one-main	moderate
region	moderate
target-size	serious

[adobe-bot]

Accessibility Violations Found

Id	Impact
label-title-only	serious
landmark-one-main	moderate
region	moderate
target-size	serious

[adobe-bot]

Accessibility Violations Found

Id	Impact
label-title-only	serious
landmark-one-main	moderate
region	moderate
target-size	serious

[codecov]

Codecov Report

Attention: Patch coverage is 85.71429% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 81.30%. Comparing base (2b0116c) to head (e01dc88).

Files	Patch %	Lines
...s/internal/form/FormConfigurationProviderImpl.java	85.71%	0 Missing and 2 partials ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##                dev    #1223      +/-   ##
============================================
+ Coverage     81.27%   81.30%   +0.02%     
- Complexity      807      812       +5     
============================================
  Files            93       94       +1     
  Lines          2168     2182      +14     
  Branches        294      298       +4     
============================================
+ Hits           1762     1774      +12     
  Misses          252      252              
- Partials        154      156       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[rismehta]

check comments

[rismehta]

the patch coverage is less, please fix the coverage

[adobe-bot]

Lighthouse scores (desktop)

	Performance	Accessibility	Best-Practices	SEO
Scores	100	96	96	75

[adobe-bot]

Lighthouse scores (mobile)

	Performance	Accessibility	Best-Practices	SEO
Scores	98	96	96	75

[adobe-bot]

Accessibility Violations Found

Id	Impact
label-title-only	serious
landmark-one-main	moderate
region	moderate
target-size	serious

[adobe-bot]

Accessibility Violations Found

Id	Impact
label-title-only	serious
landmark-one-main	moderate
region	moderate
target-size	serious

[adobe-bot]

Accessibility Violations Found

Id	Impact
label-title-only	serious
landmark-one-main	moderate
region	moderate
target-size	serious

[adobe-bot]

Accessibility Violations Found

Id	Impact
label-title-only	serious
landmark-one-main	moderate
region	moderate
target-size	serious

[deepprakash345]

the patch coverage is less, please fix the coverage

@rismehta , as seen in the screenshot of the codecov, all the branches of the code are covered, its showing the ifs as partial lines.

[jalagari]

the patch coverage is less, please fix the coverage

@rismehta , as seen in the screenshot of the codecov, all the branches of the code are covered, its showing the **ifs** as partial lines.

@deepprakash345 - Can we add unit test cases for missing cases?

[rismehta]

Are the FT present in the test case, dependent on the feature which is part of this PR ? If not, please remove these from the CL

[adobe-bot]

Lighthouse scores (mobile)

	Performance	Accessibility	Best-Practices	SEO
Scores	92	96	100	75

[adobe-bot]

Lighthouse scores (desktop)

	Performance	Accessibility	Best-Practices	SEO
Scores	100	96	100	75

[adobe-bot]

Accessibility Violations Found

Id	Impact
label-title-only	serious
landmark-one-main	moderate
region	moderate
target-size	serious

[adobe-bot]

Accessibility Violations Found

Id	Impact
label-title-only	serious
landmark-one-main	moderate
region	moderate
target-size	serious

[adobe-bot]

Accessibility Violations Found

Id	Impact
label-title-only	serious
landmark-one-main	moderate
region	moderate
target-size	serious

[deepprakash345]

Are the FT present in the test case, dependent on the feature which is part of this PR ? If not, please remove these from the CL

removed

[deepprakash345]

the patch coverage is less, please fix the coverage

@rismehta , as seen in the screenshot of the codecov, all the branches of the code are covered, its showing the **ifs** as partial lines.

@deepprakash345 - Can we add unit test cases for missing cases?

added UTs , coverage check passed.

[deepprakash345]

@rismehta , @jalagari please check

[adobe-bot]

Lighthouse scores (desktop)

	Performance	Accessibility	Best-Practices	SEO
Scores	100	96	100	75

[adobe-bot]

Lighthouse scores (mobile)

	Performance	Accessibility	Best-Practices	SEO
Scores	94	96	100	75

[adobe-bot]

Accessibility Violations Found

Id	Impact
label-title-only	serious
landmark-one-main	moderate
region	moderate
target-size	serious

[adobe-bot]

Accessibility Violations Found

Id	Impact
label-title-only	serious
landmark-one-main	moderate
region	moderate
target-size	serious

[adobe-bot]

Accessibility Violations Found

Id	Impact
label-title-only	serious
landmark-one-main	moderate
region	moderate
target-size	serious

[adobe-bot]

Lighthouse scores (mobile)

	Performance	Accessibility	Best-Practices	SEO
Scores	93	96	100	75

[adobe-bot]

Lighthouse scores (desktop)

	Performance	Accessibility	Best-Practices	SEO
Scores	100	96	100	75

[adobe-bot]

Accessibility Violations Found

Id	Impact
label-title-only	serious
landmark-one-main	moderate
region	moderate
target-size	serious

[adobe-bot]

Accessibility Violations Found

Id	Impact
label-title-only	serious
landmark-one-main	moderate
region	moderate
target-size	serious

[adobe-bot]

Accessibility Violations Found

Id	Impact
label-title-only	serious
landmark-one-main	moderate
region	moderate
target-size	serious

Comments incorporated