Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong URLs in dashboard #310

Closed
samuel-clara opened this issue Aug 2, 2024 · 9 comments
Closed

Wrong URLs in dashboard #310

samuel-clara opened this issue Aug 2, 2024 · 9 comments
Labels
bug Something isn't working

Comments

@samuel-clara
Copy link

Describe the bug

The URLS of the href attribute in the "404 File Not Found URL" column are wrong. Site slug is applied two times.
The text shows the right URL

To reproduce

Steps to reproduce the behaviour:

  1. Go to Dashboard
  2. Hover one of the links of the "404 File Not Found URL" column
  3. URL differs from text

Expected behaviour

A clear and concise description of what you expected to happen.

Screenshots

meteorit 2024-08-02 at 17 13 56@2x

Versions

  • Plugin version: 4.1.18
  • Craft version: 4.10.7
@samuel-clara samuel-clara added the bug Something isn't working label Aug 2, 2024
@khalwat
Copy link
Contributor

khalwat commented Aug 5, 2024

hmmmm. This is an issue that should have been fixed in a recent release of the plugin:

https://github.com/nystudio107/craft-retour/releases/tag/4.1.17

See this issue:

#290

Is there anything you can provide me to help narrow down the issue?

@white-lukas
Copy link

Bumping into the same thing running 5.0.2 and craft 5.3.1

@khalwat
Copy link
Contributor

khalwat commented Aug 8, 2024

Can you give me some more information so I can attempt to replicate this here?

How are your sites set up in terms of URL structure, and what URLs are you seeing vs. what you're expecting to see? And are you seeing this in the Dashboard, or in the Redirects views?

@white-lukas
Copy link

Sites are set up with urls
@web
@web/fr
...

It's showing in the dashboard view, and then the 2 subpaths are both added

@francoislevesque
Copy link

Hey there, we have the same issue on our end.

Sites are set up like that :
@web
@web/en

On my dashboard I see this :
image

When clicking on /en/immep5/blog379/, it opens /en/en/immep5/blog379.

The site slug is doubled for some reason 🤔

Thank you for your time and support !

@khalwat
Copy link
Contributor

khalwat commented Aug 17, 2024

Okay, I figured out why this was "fixed" already, but I was unable to replicate it. I was testing it in an environment where the "main" site had no path prefix at all, e.g.: http://example.com/, in which case, it would indeed work as expected.

However if the primary site had a prefix, it would fail. I've addressed it in the above commits.

Please try it out as indicated below, to ensure that it has fixed it for your particular setup, and let me know:

Craft CMS 3:

You can try it now by setting your semver in your composer.json to look like this:

    "nystudio107/craft-retour": "dev-develop as 3.2.19”,

Then do a composer clear-cache && composer update

…..

Craft CMS 4:

You can try it now by setting your semver in your composer.json to look like this:

    "nystudio107/craft-retour": "dev-develop-v4 as 4.1.19”,

Then do a composer clear-cache && composer update

…..

Craft CMS 5:

You can try it now by setting your semver in your composer.json to look like this:

    "nystudio107/craft-retour": "dev-develop-v5 as 5.0.3”,

Then do a composer clear-cache && composer update

@khalwat khalwat closed this as completed Aug 17, 2024
khalwat added a commit that referenced this issue Aug 17, 2024
…g when clicked on in some multi-site setups ([#310](#310))
khalwat added a commit that referenced this issue Aug 17, 2024
…g when clicked on in some multi-site setups ([#310](#310))
khalwat added a commit that referenced this issue Aug 18, 2024
…g when clicked on in some multi-site setups ([#310](#310))
@khalwat
Copy link
Contributor

khalwat commented Aug 19, 2024

@samuel-clara @francoislevesque @white-lukas let me know how you go with the above ^

@francoislevesque
Copy link

@khalwat On my end, it seems to do the trick.
The URLs are now fixed, thank you Andrew !

@samuel-clara
Copy link
Author

@khalwat "dev-develop-v4 as 4.1.19" fixed this on my end. Thank you so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants