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

SM optimisation #251

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

SM optimisation #251

wants to merge 31 commits into from

Conversation

lounres
Copy link
Member

@lounres lounres commented Jul 8, 2020

Made #249. Useful links:

Важно

В метаданных изображения нужно указывать с абсолютными ссылками. Поэтому я сделал "компилятор" метаданных и файла robots.txt (о нём лучше спрашивать отдельно; пока он ничего не делает особого) - python-скрипт. Небольшие условия его использования можно прочитать в нём самом. Но его обязательно нужно запускать перед выкладыванием на прод. Предложения по оптимизации процесса уточнения ссылок приветствуются. Правильную ссылку я укажу как только изменения будут одобрены.

lounres added 4 commits July 8, 2020 12:54
Added description tag ang OG tags in index.html .
Created robot.txt .
Made link compiler for robots.txt and OG tags.
Pair of little changes in descriptions.
@lounres lounres added enhancement New feature or request front Everything that is connected only with front part labels Jul 8, 2020
@lounres lounres self-assigned this Jul 8, 2020
@lounres lounres marked this pull request as draft July 8, 2020 13:57
@lounres

This comment has been minimized.

@lounres lounres linked an issue Jul 8, 2020 that may be closed by this pull request
@lounres

This comment has been minimized.

@lounres
Copy link
Member Author

lounres commented Jul 11, 2020

Результаты

К сожалению, не получилось идеально подстроиться под некоторых (Android-версии, Telegram, WhatsApp). Но вот, что есть:

  • ВК: сообщение и пост на ПК.
    изображение
    изображение

  • ВК: сообщение и пост на Android.
    изображение
    изображение

  • Facebook: сообщение и пост на ПК.
    изображение
    изображение

  • Facebook: сообщение и пост на Android.
    изображение
    изображение

  • Twitter: сообщение и пост на ПК.
    изображение
    изображение

  • Twitter: сообщение и пост на Android.
    изображение
    изображение
    изображение

  • Telegram: сообщение на ПК и на Android.
    изображение
    изображение

  • WhatsApp: сообщение на ПК и на Android.
    изображение
    изображение

lounres added 2 commits July 11, 2020 21:01
Updated docs in SM_links_compiler.py
@lounres lounres marked this pull request as ready for review July 11, 2020 18:20
Copy link
Member

@alexdany657 alexdany657 left a comment

Choose a reason for hiding this comment

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

Круто!
Однако есть две проблемы:

  1. В meta тэгах указыввется абсолютный путь, что не хорошо, так как у нас несколько разных url-ов (prod, staging и личные).
  2. Есть необходимость вызывать скрипт (собирать) перед выкатыванием. Это не очень хорошо, так как что-то может пойти не так, и выкатывание затянется. Также, в скрипте пути относительные, что может привести к неприятным последствиям при вызове неоттуда.

Надо разобраться с template-ами/активными страницами и генерить meta-тэги при помощи них.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request front Everything that is connected only with front part
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Social Media optimisation
2 participants