search

aemsites / 888de

Apache License 2.0
0 stars 0 forks source link

Inner pages: Discrepancies in the 404 page #188

Closed jeric888williamhill closed 2 months ago

jeric888williamhill commented 2 months ago

Hi @helms-charity,

I have 2 issues on this page

Please see bug details below:

1st issue: Incorrect redirection of the CTAs

Steps to Reproduce:

  1. Access 404 page. (e.g https://eds-www.888.de/123/)
  2. Click on the CTA of 888slots, 888poker, and 888sport.
  3. Notice that the CTA redirects to Canadian site.
image

**2nd issue:** The label of 888poker and 888sport CTA is incorrect

Steps to Reproduce:

  1. Access 404 page. (e.g https://eds-www.888.de/123/)
  2. Notice that the label of 888poker and 888sport CTA is incorrect. It should be "888poker herunterladen" for 888poker and "Bei 888sport anmelden" for 888sport as compared to original site.

image

helms-charity commented 2 months ago

Hi Jeric, it seems this was only a content authoring issue, please test it again when you have a chance.

jeric888williamhill commented 2 months ago

HI @helms-charity,

New issue found.

The CTA opens a new tab instead of staying in the same tab.

image

Please also redirect the CTAs based on the URL given in the original site. 888slots: https://www.888slots.de/registrieren/ 888poker: https://www.888poker.de/download-poker/ 888sport: https://www.888sport.de/?deeplink=opennrs

dave-fink commented 2 months ago

The links have been updated. Please revalidate.

jeric888williamhill commented 2 months ago

Still incorrect redirection and opens a new tab for 888sport

image

jeric888williamhill commented 2 months ago

Hi @dave-fink,

Aside from the existing issue above, I would like to add this issue that happens in mobile devices.

Issue: The creative for the 404 page is too small compared with the original site making the word "oh je" overlaps with the 888 flag in the creatives. See screenshot below.

image

kailasnadh790 commented 2 months ago

Styling issue is fixed. Please validate.

Issue with button redirections are related to https://github.com/aemsites/888de/issues/191#issuecomment-2043316748 .

jeric888williamhill commented 2 months ago

Fixed verified for styling issue.

Open issue: No redirection link for all CTA. Need to wait for @rafim888 to address the issue in #191

kailasnadh790 commented 2 months ago

@jeric888williamhill Redirections are fixed. #191 is kept open to address GA events. I guess, we can close this ticket after validation.

jeric888williamhill commented 2 months ago

@kailasnadh790, CTAs in the 404 page still not clickable from my end.

kailasnadh790 commented 2 months ago

@jeric888williamhill I have seen this issue with previously accessed error page URL...some kind of caching at 888 CDN is causing this. Can you try with a new "wrong URL" ?

https://github.com/aemsites/888de/assets/53608019/dac4928e-c43b-4d39-acb1-7c6d6e4a1da9

jeric888williamhill commented 2 months ago

Hi @kailasnadh790, I see. Thanks for explaining it, now it works. Fixed/verified for this issue. Closing this now.