Skip to content
This repository has been archived by the owner on Feb 23, 2020. It is now read-only.

rf#140 Change redirects dep #231

Closed
wants to merge 2 commits into from
Closed

rf#140 Change redirects dep #231

wants to merge 2 commits into from

Conversation

duker33
Copy link
Contributor

@duker33 duker33 commented Jun 22, 2018 •
edited
Loading

fidals/refarm-site#140

Code depends on refarm-site feature. So, this PR merging is blocked by fidals/refarm-site#141, but is ready for review

@duker33 duker33 self-assigned this Jun 22, 2018
@duker33 duker33 requested a review from ArtemijRodionov June 22, 2018 13:56
ArtemijRodionov
ArtemijRodionov suggested changes Jun 27, 2018
View reviewed changes
Copy link
Contributor

@ArtemijRodionov ArtemijRodionov left a comment

Choose a reason for hiding this comment

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

You have to update the redirect middleware, isn't it?

@duker33
Copy link
Contributor Author

duker33 commented Jul 3, 2018

i'll create separated pr with separated task

@duker33 duker33 closed this Jul 3, 2018
@duker33 duker33 added the invalid label Jul 3, 2018
@duker33 duker33 deleted the rf140_redirects_dep branch July 5, 2018 13:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ArtemijRodionov ArtemijRodionov ArtemijRodionov requested changes

Assignees

@duker33 duker33

Labels
invalid
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@duker33 @ArtemijRodionov