getdokan / dokan-wpml

WPML compatibility with dokan plugin
5 stars 11 forks source link

⚠️⚠️ SELLER REGISTRATION FORM > Shop url > Translated forms > double // #59

Closed Sophie-2e closed 4 months ago

Sophie-2e commented 8 months ago

I notice an issue with the shop url in the vendor registration form www.website.com/shop/name-shop-vendor in other languages where a double // is added to the store url such as www.website.com/fr//shop/name-shop-vendor or www.website.com/nl//shop/name-shop-vendor. Please review this image:

Screenshot 2024-03-30 at 11 18 45
Mohaiminulislam1989 commented 8 months ago

@shams-sadek1981 vai, Please assign this to the developer

Sophie-2e commented 6 months ago

PS: @shams-sadek1981 still bugging with Dokan & Dokan Pro 3.11.0 when will this be fixed?

Sophie-2e commented 5 months ago

So when this is going to be fixed? Still Buging with Dokan 3.11.2 and Pro 3.11.1 It might just be a small error in the code that is adding a / Please update me thanks

@shams-sadek1981 @Mohaiminulislam1989

Sophie-2e commented 4 months ago

Hello @shohag121 @MdAsifHossainNadim @mrabbani hope you're well. Just tested the fix released in Dokan 3.11.4 : 'Shop URL rendered double slash when using WPML on vendor registration' does not work. Still does not work even with Dokan WPML Integration Version 1.1.5:

Screenshot 2024-07-17 at 17 27 52

Reopen GitHub here: https://github.com/getdokan/dokan-wpml/issues/79

MdAsifHossainNadim commented 4 months ago

@Sophie-2e The problem does not reproduce on my end. This could be a cache issue. Please resync your strings, clear the cache and test it again. Then, please provide us with a screencast if you can reproduce this issue.

Screenshot at Jul 25 14-00-14

Sophie-2e commented 4 months ago

@MdAsifHossainNadim no cache issue, no sync issue either as I already did these manipulations many times since I updated. You will see proof in these videos:

Are you sure to test with the shortcode: [dokan-vendor-registration] ? Screenshot 2024-07-25 at 12 00 56

Sophie-2e commented 4 months ago

@MdAsifHossainNadim @Mohaiminulislam1989 No solution does not work, review the comment here https://github.com/getdokan/dokan/pull/2319#issuecomment-2262719788

Mohaiminulislam1989 commented 4 months ago

@MdAsifHossainNadim @Mohaiminulislam1989 No solution does not work, review the comment here getdokan/dokan#2319 (comment)

@MdAsifHossainNadim please check