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.

Sign up for GitHub

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

Jump to bottom

Redirects mech broken #360

Closed
duker33 opened this issue Jun 16, 2018 · 5 comments
Closed

Redirects mech broken #360

duker33 opened this issue Jun 16, 2018 · 5 comments
Assignees
@duker33
Labels
1 hour typical issue size. It's one pdd hour 1 burning issue blocker this issue blocks another issue bug report Put it to completed tasks temporary list for business

Comments

@duker33
Copy link
Contributor

duker33 commented Jun 16, 2018

Prod db contains these urls in redirects list.
But redirect from first to second is not working

  1. https://www.shopelectro.ru/catalog/categories/akkumuliatory-270/tags/6f22/
  2. https://www.shopelectro.ru/catalog/categories/tiporazmer-6f22/
@duker33 duker33 added bug 1 hour typical issue size. It's one pdd hour 1 burning issue blocker this issue blocks another issue labels Jun 16, 2018
@duker33 duker33 self-assigned this Jun 16, 2018
@duker33 duker33 mentioned this issue Jun 16, 2018
Closed
@duker33 duker33 changed the title Tags mech broken Redirects mech broken Jun 18, 2018
@duker33
Copy link
Contributor Author

duker33 commented Jun 18, 2018

django standard redirects mech triggers only for not found urls
https://github.com/django/django/blob/master/django/contrib/redirects/middleware.py#L26

I'll try to move this app to refarm-site and customize it's behaviour for our needs

@duker33
Copy link
Contributor Author

duker33 commented Jun 22, 2018

we'll resolve this problem at fidals/refarm-site#140

Just created the test. It specifies problem

duker33 added a commit that referenced this issue Jun 22, 2018
@duker33
#360 Refresh py reqs according to stb
829350b
duker33 added a commit that referenced this issue Jun 22, 2018
@duker33
#360 Create test with unexpected behaviour explaining
f0e5567
@duker33 duker33 mentioned this issue Jun 22, 2018
Merged
@duker33 duker33 added the review waits for review label Jun 22, 2018
duker33 added a commit that referenced this issue Jun 22, 2018
@duker33
#360 Fix commented test and refarm-site dependency
a4372d5
duker33 added a commit that referenced this issue Jun 22, 2018
@duker33
#360 Fix commented test and refarm-site dependency
ca151d5
duker33 added a commit that referenced this issue Jun 22, 2018
@duker33
#360 Fix commented test and refarm-site dependency
a3c499b
duker33 added a commit that referenced this issue Jun 25, 2018
@duker33
#360 Skip expected failure
8fc3b6f
@duker33 duker33 added refine issue already have resolving code, but code needs some fixes and removed review waits for review labels Jul 3, 2018
duker33 added a commit that referenced this issue Jul 3, 2018
@duker33
#360 Refresh py reqs according to stb
6eeadc4
duker33 added a commit that referenced this issue Jul 3, 2018
@duker33
#360 Create test with unexpected behaviour explaining
933bc0e
duker33 added a commit that referenced this issue Jul 3, 2018
@duker33
#360 Fix commented test and refarm-site dependency
df447a7
duker33 added a commit that referenced this issue Jul 3, 2018
@duker33
#360 Review#1. Rm several asserts
4636a94
duker33 added a commit that referenced this issue Jul 3, 2018
@duker33
#360 Refresh py reqs according to stb
5c602ba
duker33 added a commit that referenced this issue Jul 3, 2018
@duker33
#360 Create test with unexpected behaviour explaining
c4ee94c
duker33 added a commit that referenced this issue Jul 3, 2018
@duker33
#360 Fix commented test and refarm-site dependency
e393db9
duker33 added a commit that referenced this issue Jul 3, 2018
@duker33
#360 Skip expected failure
a31dffe
duker33 added a commit that referenced this issue Jul 3, 2018
@duker33
#360 Review#1. Rm several asserts
7b80834
duker33 added a commit that referenced this issue Jul 3, 2018
@duker33
#360 Review#2. Names and some reqs
28a932e
duker33 added a commit that referenced this issue Jul 5, 2018
@duker33
#360 Review#3. Rm redundant file open
f9bb0c4
duker33 added a commit that referenced this issue Jul 5, 2018
@duker33
#360 Skip problem test
951054e
duker33 added a commit that referenced this issue Jul 5, 2018
@duker33
#360 Skip problem test
df2b032
duker33 added a commit that referenced this issue Jul 5, 2018
@duker33
#360 Skip problem test
5b3ae38
duker33 added a commit that referenced this issue Jul 5, 2018
@duker33
#360 Lint's minor fix
8085903
@duker33 duker33 added the review waits for review label Jul 6, 2018
duker33 added a commit that referenced this issue Jul 7, 2018
@duker33
#360 Test for wrong redirects behaviour (#381)
daee6d9 
- Refresh py reqs according to stb
- Create test with unexpected behaviour explaining
- Fix commented test and refarm-site dependency
- Skip expected failure
- Review#1. Rm several asserts
- Review#2. Names and some reqs
- Skip problem test
- Skip problem test
- Lint's minor fix
This was referenced Jul 7, 2018
Closed
Closed
Closed
@0pdd
Copy link
Collaborator

0pdd commented Jul 7, 2018

@duker33 3 puzzles #405, #406, #407 are still not solved.

@duker33 duker33 added report Put it to completed tasks temporary list for business and removed refine issue already have resolving code, but code needs some fixes review waits for review labels Jul 7, 2018
@0pdd
Copy link
Collaborator

0pdd commented Jul 20, 2018

@duker33 the puzzle #407 is still not solved; solved: #405, #406.

@0pdd
Copy link
Collaborator

0pdd commented Jul 23, 2018

@duker33 all 3 puzzles are solved here: #405, #406, #407.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@duker33 duker33

Labels
1 hour typical issue size. It's one pdd hour 1 burning issue blocker this issue blocks another issue bug report Put it to completed tasks temporary list for business
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@duker33 @0pdd