Skip to content
This repository has been archived by the owner on May 26, 2021. It is now read-only.

#35 add request url to stub #43

Merged
merged 7 commits into from
Oct 17, 2020

Conversation

PetrS12
Copy link
Contributor

@PetrS12 PetrS12 commented Oct 11, 2020

closes #35

@lowitea lowitea self-requested a review October 11, 2020 23:35

@register.simple_tag(takes_context=True, name='absolute')
def get_absolute_url_tag(context, url):
"""Тег возвращающий абсолютный урл."""
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you please write the docstring in English?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry for my Russian;) It's just a habit.
Corrected.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thank you)

@codecov
Copy link

codecov bot commented Oct 15, 2020

Codecov Report

Merging #43 into develop will increase coverage by 0.10%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #43      +/-   ##
===========================================
+ Coverage    98.21%   98.31%   +0.10%     
===========================================
  Files            4        5       +1     
  Lines          112      119       +7     
  Branches         4        4              
===========================================
+ Hits           110      117       +7     
  Misses           2        2              
Impacted Files Coverage Δ
http_stubs/templatetags/stub_tags.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 82b43f9...2ab87c6. Read the comment docs.

@@ -0,0 +1,7 @@
{% extends "admin/change_form.html" %}
{% load i18n admin_urls static admin_modify simpletags%}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
{% load i18n admin_urls static admin_modify simpletags%}

Здесь не используется ни один тег (кроме доступных по умолчанию).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks! Corrected.

{{ field.field }}
{# added request url for path field(issue #35) #}
{% if field.field.name == 'path' %}<div class="readonly"><div id="full-path">{% absolute field.field.value %}</div><div id="path-copied-alert">Скопировано!</div></div>{% endif %}
{# #}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
{# #}

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Corrected.

Copy link
Collaborator

Choose a reason for hiding this comment

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

BTW, I've missed Russian word 'Скопировано!' here. :) Probably should be in English or use i18n for translation.

Copy link
Contributor

Choose a reason for hiding this comment

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

Meeh... I has created a new issue for it #50

@lowitea lowitea added this to the 0.1.3 milestone Oct 17, 2020
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@lowitea lowitea merged commit 867e290 into Uma-Tech:develop Oct 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a request url to the stub edit page
3 participants