urfuwo / hlx-test

Apache License 2.0
0 stars 3 forks source link

fix: tag list links for news #524

Closed mhaack closed 2 months ago

mhaack commented 2 months ago

Relates to #391

Test URLs: Content Hub: Before: https://main--hlx-test--urfuwo.hlx.live/blog/2023/11/idc-marketscape-sap-a-leader-cloud-enabled-sourcing-applications After: https://391-fix-tags--hlx-test--urfuwo.hlx.live/blog/2023/11/idc-marketscape-sap-a-leader-cloud-enabled-sourcing-applications

aem-code-sync[bot] commented 2 months ago

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed. In case there are problems, just click a checkbox below to rerun the respective action.

Commits * [aa7e005](https://github.com/urfuwo/hlx-test/commit/aa7e0059c7978bd7a4c69c7dfc174d28ef67a428) :white_check_mark: (latest) * [3dce31b](https://github.com/urfuwo/hlx-test/commit/3dce31b441d96e43ff655e33b73848d807453ab2) :white_check_mark:
aem-code-sync[bot] commented 2 months ago
Page Scores Audits Google
/blog/2023/11/idc-marketscape-sap-a-leader-cloud-enabled-sourcing-applications PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
saurabh-khare commented 2 months ago

What if a tag has both news path and topic path? For news do we show topic path and only show news path if theres no topic path?

mhaack commented 2 months ago

What if a tag has both news path and topic path? For news do we show topic path and only show news path if theres no topic path?

Yes. We have two cases for tags with both L2 page paths: