Skip to content

feat: add qs-dispatch and qs-smartphone-pro bridge support - #36

Open
devayee wants to merge 7 commits into
ProdigyPRP:mainfrom
devayee:main
Open

feat: add qs-dispatch and qs-smartphone-pro bridge support#36
devayee wants to merge 7 commits into
ProdigyPRP:mainfrom
devayee:main

Conversation

@devayee

@devayee devayee commented Aug 3, 2026

Copy link
Copy Markdown

Adds bridge support for two Quasar scripts:

  • qs-dispatch — new dispatch adapter (modules/dispatch/qs-dispatch/server.lua)
  • qs-smartphone-pro — new phone adapter (modules/phone/qs-smartphone-pro/server.lua)

@PsychoShock PsychoShock left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This needs a better review since some things seems not to be on documentation and doubt it works unless doc is not updated.

function phone.sendNotification(src, title, content)
-- Source: https://www.quasar-store.com/docs/smartphone
local identifier = GetPlayerIdentifier(src, 0)
local phoneNumber = exports["qs-smartphone-pro"]:GetPhoneNumberFromIdentifier(identifier, false)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is it tested? Documentation seems to be

Suggested change
local phoneNumber = exports["qs-smartphone-pro"]:GetPhoneNumberFromIdentifier(identifier, false)
local phoneNumber = exports['qs-smartphone']:GetCurrentPhoneNumber(src)

From https://www.quasar-store.com/docs/smartphone/developer-api. I dont even see this export/function,

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Turns out Quasar actually removed/restructured the docs for Smartphone Pro at some point, so I was working off pages that no longer reflect the current export set. But everything is tested and working, if needed I can edit it with current exports

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Tested and confirmed one more time. GetPhoneNumberFromIdentifier only valid and confirmed export for Smartphone PRO not Smartphone default, this is PRO version. GetCurrentPhoneNumber export is working only on default versions. GetPhoneNumberFromIdentifier test
image

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants