Skip to content

play: extend error message #1171

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

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

Conversation

patapenka-alexey
Copy link
Contributor

@patapenka-alexey patapenka-alexey commented Jun 6, 2025

This patch extends error message if space to play data is unavailable or user does not have permission to work with it, because the net.box module does not have means to distinguish between these errors.

Closes #TNTP-2366

@patapenka-alexey patapenka-alexey force-pushed the patapenka-alexey/tntp-2366-tt-play-no-space-id branch from 351b700 to 30dad5b Compare June 6, 2025 11:54
@patapenka-alexey patapenka-alexey changed the title play: update error message play: extend error message Jun 6, 2025
@patapenka-alexey patapenka-alexey marked this pull request as ready for review June 6, 2025 12:32
@patapenka-alexey patapenka-alexey added the full-ci Enables full ci tests label Jun 6, 2025
Copy link
Contributor

@oleg-jukovec oleg-jukovec left a comment

Choose a reason for hiding this comment

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

Please, describe in the commit and PR message why we could not resolve the problem (I guess we couldn't detect a specific error someway).

It would be nice if you add a comment into the code about it, up to you.

This patch extends error message if space to play data is unavailable
or user does not have permission to work with it, because the net.box
module does not have means to distinguish between these errors.

Closes #TNTP-2366
@patapenka-alexey patapenka-alexey force-pushed the patapenka-alexey/tntp-2366-tt-play-no-space-id branch from 30dad5b to 04983d1 Compare June 9, 2025 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
full-ci Enables full ci tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants