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

Возможность скопировать комментарий в виде Markdown-кода #885

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

uburuntu
Copy link

Реализация к обсуждению #874.

Есть старый комментарий:

Screenshot 2022-01-27 at 14 38 02

Так выглядит попытка его отредактировать:

Screenshot 2022-01-27 at 14 38 17

Предлагается сделать так, чтоб оставалась возможность скопировать свой старый коммент:

Screenshot 2022-01-27 at 16 07 02

Что произошло:

  • теперь нет AccessDenied при простом открытии окна комментария, убрал кнопку Сохранить
  • хацкеры при попытке отправить POST запрос всё ещё получат AccessDenied

@uburuntu uburuntu requested a review from vas3k as a code owner January 27, 2022 16:43
@uburuntu
Copy link
Author

Есть конфликт с #799 :) можно будет там не убирать ссылку, а просто тултип выводить

@devcooch
Copy link
Collaborator

Может лучше по аналогии с экспортом поста в md просто добавить отдельный урл для экспорта?

@uburuntu
Copy link
Author

Может лучше по аналогии с экспортом поста в md просто добавить отдельный урл для экспорта?

Что-то я пропустил эту фичу, где её найти?

@devcooch
Copy link
Collaborator

club/urls.py:

path("<slug:post_type>/<slug:post_slug>.md", md_show_post, name="md_show_post")

posts/api.py - md_show_post

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.

2 participants