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.

Sign up for GitHub

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

Jump to bottom

%location% is[(n't| not) [with]in radius %number% (around|of) %location #5201

Closed
1 task done
JakeGBLP opened this issue Nov 11, 2022 · 1 comment
Closed
1 task done

%location% is[(n't| not) [with]in radius %number% (around|of) %location #5201

JakeGBLP opened this issue Nov 11, 2022 · 1 comment
Labels
completed The issue has been fully resolved and the change will be in the next Skript update. enhancement Feature request, an issue about something that could be improved, or a PR improving something. priority: lowest "Nice to have" updates that are not required (tiny low impact bug fixes or QoL enhancements).

Comments

@JakeGBLP
Copy link

JakeGBLP commented Nov 11, 2022 •
edited
Loading

Suggestion

Add is within radius condition.

Why?

Would be a cool condition to have, would make code cleaner when checking that scenario.

Other

No response

Agreement

  • I have read the guidelines above and affirm I am following them with this suggestion.
@AyhamAl-Ali AyhamAl-Ali added enhancement Feature request, an issue about something that could be improved, or a PR improving something. priority: lowest "Nice to have" updates that are not required (tiny low impact bug fixes or QoL enhancements). labels Nov 11, 2022
@bluelhf
Copy link
Contributor

bluelhf commented Nov 14, 2022

Instead of radius %number%, could we use something like %number% (block|metre|meter)[s]?

@UnderscoreTud UnderscoreTud mentioned this issue Nov 14, 2022
Merged
@AyhamAl-Ali AyhamAl-Ali added the PR available Issues which have a yet-to-be merged PR resolving it label Nov 15, 2022
@TheLimeGlass TheLimeGlass added completed The issue has been fully resolved and the change will be in the next Skript update. and removed PR available Issues which have a yet-to-be merged PR resolving it labels Nov 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
completed The issue has been fully resolved and the change will be in the next Skript update. enhancement Feature request, an issue about something that could be improved, or a PR improving something. priority: lowest "Nice to have" updates that are not required (tiny low impact bug fixes or QoL enhancements).
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@TheLimeGlass @AyhamAl-Ali @APickledWalrus @bluelhf @JakeGBLP