Skip to content
This repository has been archived by the owner on Dec 3, 2024. It is now read-only.

I added FireFor which fires a remote of an array of players, I did not test it though. #2

Closed
wants to merge 2 commits into from
Closed

I added FireFor which fires a remote of an array of players, I did not test it though. #2

wants to merge 2 commits into from

Conversation

Daemon6109
Copy link

@Daemon6109 Daemon6109 commented Sep 9, 2024 •
edited
Loading

Summary

Checks

By submitting your pull request for review, you agree to the following:

  • I give full permission for maintainers and contributors of this repository to change my code in any way.
  • I am open to this pull request being a part of a completely open source repository which can be modified at any time, and I abide by the license terms.
  • To the best of my knowledge, there are no issues present in this pull request.

@Daemon6109
Update Types.luau
8e31030 
Added FireFor.
@Daemon6109
Update Server.luau
3a9cc62 
Added FireFor.
@2jammers
Copy link
Member

Going to deny this because the Fire methods are for mainly general use, and this is too specific. The best practice would to make your own loop and call :Fire

@2jammers 2jammers closed this Sep 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Daemon6109 @2jammers