Skip to content
This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

feat(server): bub-mdt compatibility #7

Merged
merged 7 commits into from
Dec 31, 2024
Merged

Conversation

its4lion
Copy link
Contributor

Pull Request Description

Added support for bub-mdt dispatch. (Not tested in-game)

Major Changes

Modifications done:

  • 🍃 New Feature
  • ♻️ Refactor
  • 🐛 Bug Fix
  • 🎨 Design
  • 🔥 Optimization
  • 🎒 Other

@Maximus7474
Copy link
Owner

Hi Lion,

Thanks for your contribution, however when lookign through the source code for bub mdt I noticed that the event you've integrated doesn't seem to be the recommended method to create calls, it rather gets scripts to create calls using a server export.
I'll add a quick structure and test it to your branch that will provide support to handling exports for that ressource.

server/bub-mdt.lua Outdated Show resolved Hide resolved
Copy link
Owner

@Maximus7474 Maximus7474 left a comment

Choose a reason for hiding this comment

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

Needs clarifications on certain implementations.

@Maximus7474 Maximus7474 added the enhancement New feature or request label Dec 30, 2024
@Maximus7474
Copy link
Owner

I'll let you resolve the last comment and update the README and it should be good to merge.

@its4lion its4lion requested a review from Maximus7474 December 31, 2024 03:43
@its4lion
Copy link
Contributor Author

I’ve made the necessary changes and updated the README as requested.
Apologies for any mistakes. I'm still learning how pull requests work. Thanks for your patience!

@Maximus7474
Copy link
Owner

All good, everyone has to learn one way or another !

@Maximus7474 Maximus7474 merged commit 0b34615 into Maximus7474:main Dec 31, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants