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

sol #468

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

sol #468

wants to merge 2 commits into from

Conversation

Familenko
Copy link

No description provided.

@Familenko
Copy link
Author

Familenko commented Nov 16, 2023

Uploading Знімок екрана 2023-11-16 о 18.29.22.png…
Знімок екрана 2023-11-16 о 17 34 25
Знімок екрана 2023-11-16 о 17 34 18
Знімок екрана 2023-11-16 о 17 34 08
Знімок екрана 2023-11-16 о 17 34 00
Знімок екрана 2023-11-16 о 17 33 39

Choose a reason for hiding this comment

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

path("accounts/profile/", include("taxi.urls", namespace="taxi")),
Why you do this but not to specify login redirect page in settings?

@Daniil-Pankieiev
Copy link

You need to be albe go to driver detail click to nickname if you log in

Copy link

@Daniil-Pankieiev Daniil-Pankieiev left a comment

Choose a reason for hiding this comment

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

Fix problems

@Familenko
Copy link
Author

You need to be albe go to driver detail click to nickname if you log in

What makes you to thing that this option is not working?

@Familenko
Copy link
Author

Fix problems
Describe a problem, becouse this one is working You need to be albe go to driver detail click to nickname if you log in
And this one is not a problem

path("accounts/profile/", include("taxi.urls", namespace="taxi")),
Why you do this but not to specify login redirect page in settings?

This is another option

{% block content %}
<p>Logged out!</p>
<a href="{% url 'login'%}">Click here to login again.</a>
{% endblock %}

Choose a reason for hiding this comment

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

add a blank line

@@ -22,4 +7,6 @@
urlpatterns = [
path("admin/", admin.site.urls),
path("", include("taxi.urls", namespace="taxi")),
path("accounts/", include("django.contrib.auth.urls")),
path("accounts/profile/", include("taxi.urls", namespace="taxi")),

Choose a reason for hiding this comment

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

Why do you need this?

Copy link

@LiudmylaKulinchenko LiudmylaKulinchenko left a comment

Choose a reason for hiding this comment

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

Well done 😸

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

Successfully merging this pull request may close these issues.

3 participants