Skip to content

Enabling custom 404 subprojects pages #6153

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

Closed

Conversation

tapaswenipathak
Copy link
Contributor

Fixes #5557.

@stsewd
Copy link
Member

stsewd commented Sep 9, 2019

We should add a test for this change

@stsewd stsewd added Needed: tests Tests are required PR: work in progress Pull request is not ready for full review labels Sep 9, 2019
Copy link
Member

@humitos humitos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @tapaswenipathak, thanks for your contribution!

I'm not sure to follow how the changes solve the issue that we are facing. Can you explain a little more your proposed solution? In the middleware we are defining rel and subproject but we are doing nothing with them.

@humitos humitos added the Needed: more information A reply from issue author is required label Nov 19, 2019
@humitos
Copy link
Member

humitos commented Dec 9, 2019

I'm going to close this PR as stale for now. Feel free to re-open if you consider and I'll be happy to review it.

@humitos humitos closed this Dec 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needed: more information A reply from issue author is required Needed: tests Tests are required PR: work in progress Pull request is not ready for full review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Proxito: look for custom 404 pages in the parent project of subprojects
3 participants