django-analytical icon indicating copy to clipboard operation
django-analytical copied to clipboard

Add Liveinternet

Open odi1n opened this issue 2 years ago • 4 comments

Addition service Liveinternet. Several implementation options have been made as per the documentation:

  1. in the form of a single image combining the counter and the LiveInternet logo
  2. in the form of two images, one of which is a counter (transparent GIF size 1x1), and the other is the LiveInternet logo. This placement method will allow you to insert the code of the invisible counter at the beginning of the page, and the logo - where the design and content of the page allows.

odi1n avatar Sep 14 '22 18:09 odi1n

Looks good to me! If it works on your end.

Can you please add tests to make the implementation complete?

Take a look at the other modules, if you need inspiration. It's easy and quick to do.

Thank you! 👍

Kind of implemented all the wishes 😊

odi1n avatar Sep 19 '22 14:09 odi1n

Flake8 and isort found 3 minor issues, and some of your tests are failing. Can you fix that?

bittner avatar Sep 26 '22 14:09 bittner

Flake8 и isort обнаружили 3 незначительные проблемы, и некоторые из ваших тестов не работают. Вы можете это исправить?

I don't have a lot of time right now, so I can't. As soon as I'm free, if I don't forget, I'll fix it.

odi1n avatar Sep 26 '22 14:09 odi1n

I don't have a lot of time right now, so I can't. As soon as I'm free, if I don't forget, I'll fix it.

It's all good, take your time. Your contribution is good on track at the moment. Unfortunately, it's necessary to satisfy some basic code quality metrics. There's not much missing, though, really. Thank you for your understanding! Maybe set yourself a reminder. ⏱️

bittner avatar Sep 26 '22 14:09 bittner