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

Feature request: 302 redirects #64

Open
getandyjones opened this issue May 22, 2017 · 5 comments
Open

Feature request: 302 redirects #64

getandyjones opened this issue May 22, 2017 · 5 comments

Comments

@getandyjones
Copy link

Hi There,

I love your plugin, but think it would be useful if I could add my 302 redirects there too - so a dropdown to select either 301 or 302 - that way, all my redirects can be in one place - what do you think?

@Jako
Copy link
Collaborator

Jako commented May 22, 2017

This is hardcoded at the moment in https://github.com/modxcms/Redirector/blob/develop/core/components/redirector/elements/plugins/redirector.plugin.php#L74

Do you need to manage it for each redirect or as a global option for all redirects?

@getandyjones
Copy link
Author

I have a mixture of 301 and 302 redirects to manage - so Ideally I wanted to be able to choose on an ad-hoc basis - and also be able to import with CSV:
pattern,target,context,type(301 or 302) type of thing?

Also, I cannot import a csv of 301s for some reason - they all fail.
I've tried with and without the csv headings, but can't get it to work?

@Jako
Copy link
Collaborator

Jako commented May 22, 2017

Adding it for each redirect will be some work, since there has to be a database field, an input in the manager page etc.

I can't say much about the CSV import problem. Maybe a BOM in the CSV or an encoding problem.

@getandyjones
Copy link
Author

OK thanks - will check the CSVs - but they look fine to me.
How should they be formatted?
ie - do we need a title for each column?
pattern,target,context

@hugopeek
Copy link

One more vote for this enhancement, with the option to set this per redirect. My client is also requesting for 410 redirects btw (page removed).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants