hlxsites / zemax

AEM Franklin Solution for https://www.zemax.com/
https://main--zemax--hlxsites.hlx.page/
Apache License 2.0
3 stars 1 forks source link

Issue 118 profile #134

Closed shehjadkhan-ansys closed 1 year ago

shehjadkhan-ansys commented 1 year ago

Profile page phase1

Fix # 118

Test URLs:

image image

This page can only be tested after login in, please use below test user to login Alice.Example@ansys.com (Almost all permission) TestExampleContact!

Barbara.Example@zemax.com (supported OpticsViewer license) TestExampleContact!

Betty.Example@zemax.com (Super Admin, with Manage Coll TestExampleContact!

Lee.Student@zemax.com (Student access) TestExampleZGAP! (No access to create ticket button, schedule call)

Paul.Educator@zemax.com (Educator) TestExampleZGAP!

aem-code-sync[bot] commented 1 year ago

Hello, I'm Franklin Bot and I will run some test suites that validate the page speed. In case there are problems, just click the checkbox below to rerun the respective action.

aem-code-sync[bot] commented 1 year ago
Page Score PSI Audit Google
/pages/profile Lighthouse returned error: FAILED_DOCUMENT_REQUEST. Lighthouse was unable to reliably load the page you requested. Make sure you are testing the correct URL and that the server is properly responding to all requests. (Details: net::ERR_ABORTED) PSI
aem-code-sync[bot] commented 1 year ago
Page Score PSI Audit Google
/ SI FCP LCP TBT CLS PSI
aem-code-sync[bot] commented 1 year ago
Page Score PSI Audit Google
/ SI FCP LCP TBT CLS PSI
aem-code-sync[bot] commented 1 year ago
Page Score PSI Audit Google
/ SI FCP LCP TBT CLS PSI
kunwarsaluja commented 1 year ago

@shehjadkhan-ansys This looks good to me..For the future phases we can think of externalize content fragments and config values outside of code(to be authorable, not relying on code updates) which we can deal with later

kunwarsaluja commented 1 year ago

@agangam Can you please review this PR as well