Skip to content

[862kbrtd6] Updated doc about error messages when refund created. - #16

Open
SoloIana wants to merge 2 commits into
masterfrom
dev/862kbrtd6-postponed-refund-approve-check
Open

[862kbrtd6] Updated doc about error messages when refund created.#16
SoloIana wants to merge 2 commits into
masterfrom
dev/862kbrtd6-postponed-refund-approve-check

Conversation

@SoloIana

Copy link
Copy Markdown
Contributor

Обновление документации в связи с изменениями логики и сообщений, предлагаемыми в PR https://github.com/ticketscloud/ticketscloud/pull/2269

Comment threadwalkthrough/refund_requests.rst Outdated
Comment on lines +304 to +305
| NB: больше не используется, т.к. организатор может также | См. следующее сообщение ↓↓↓ |
| подтвердить возврат и по перенесённому мероприятию | |

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Мне кажется что вот это приписка NB не должна быть как минимум в этом месте. А другого места для может и не был вполне - если мы нигде так развернуто не описывает то как проходят возвраты

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Приписку я добавила только чтобы показать, почему сообщение больше не используется -- оно стало неверным утверждением, и сохранить преемственность публичного API. Если кто-то это сообщение парсил, чтобы они не искали, куда оно пропало.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Я с Женей согласен. Тут это лишнее.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

То есть фактически можно просто указать "устарело / не актуально" и всё. Я своего мнения не изменил

Comment threadwalkthrough/refund_requests.rst Outdated
Comment on lines +304 to +305
| NB: больше не используется, т.к. организатор может также | См. следующее сообщение ↓↓↓ |
| подтвердить возврат и по перенесённому мероприятию | |

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Я с Женей согласен. Тут это лишнее.

@SoloIana
SoloIana requested a review from zzzsochiApril 20, 2026 14:24
@SoloIana
SoloIanaforce-pushed the dev/862kbrtd6-postponed-refund-approve-check branch from 581c154 to 48e9445CompareApril 20, 2026 14:34
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@SoloIana@zzzsochi@EvgeniyMakhmudov