-
Notifications
You must be signed in to change notification settings - Fork 311
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
feat: add barcode scanner plugin #536
Conversation
No top level dependency changes detected. Learn more about Socket for GitHub ↗︎ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't say much about the code since I am not very familiar with android APIs but I tested it and it works fine so LGTM! and I will trust you on the iOS bits since I can't test it.
@@ -32,7 +32,7 @@ open class BuildTask : DefaultTask() { | |||
val rootDirRel = rootDirRel ?: throw GradleException("rootDirRel cannot be null") | |||
val target = target ?: throw GradleException("target cannot be null") | |||
val release = release ?: throw GradleException("release cannot be null") | |||
val args = listOf("tauri", "android", "android-studio-script"); | |||
val args = listOf("/Users/lucas/projects/tauri/plugins-workspace/examples/api/./node_modules/.bin/../../../../node_modules/.pnpm/@tauri-apps+cli@2.0.0-alpha.12/node_modules/@tauri-apps/cli/tauri.js", "android", "android-studio-script"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
local path
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this okay to add? cc @Beanow
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the callout. Let me reply on the readme.
plugins/barcode-scanner/README.md
Outdated
## Sponsors | ||
|
||
<table> | ||
<tbody> | ||
<tr> | ||
<td align="center" valign="middle"> | ||
<a href="https://crabnebula.dev/" target="_blank"> | ||
<img src="sponsors/crabnebula.svg" alt="CrabNebula" width="283" height="90"> | ||
</a> | ||
</td> | ||
<td align="center" valign="middle"> | ||
<a href="https://impierce.com/" target="_blank"> | ||
<img src="sponsors/impierce.png" alt="Impierce" width="283" height="90"> | ||
</a> | ||
</td> | ||
</tr> | ||
</tbody> | ||
</table> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(cc @amrbashir @tauri-apps/tauri-board @lorenzolewis)
In the interest of not blocking useful functionality based on a sponsors policy discussion, I'm inclined to have the discussion separately. Would appreciate a gut-check from a non-CN Board Director to be sure.
But I have one immediate check I'd like to do.
@lucasfernog as PR author can you confirm that: both CrabNebula and Impierce are aware and OK with being credited?
And to be aware Tauri was pretty much caught off guard here so there's no hard commitments on whether, how and where these credits / logos might go after a followup discussion.
For further discussion I'd like to track in https://github.com/tauri-apps/governance-and-guidance since my main devils advocate question here would be: what would a different company have to do to get the same treatment? Of getting their plugin adopted by @tauri-apps
like an official plugin, with sponsor callouts and everything. And who is responsible for future maintenance, etc.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Beanow , I've opened an issue in tauri-apps/governance-and-guidance#42 to discuss this further, pending a non-CrabNebula board member to weigh in on this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Beanow Impierce is aware that we are credited but wouldn't complain at all if it is removed. Our goal is not too show up in the credits, but rather sponsor this development without any particular conditions ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@JelleMillenaar that is awesome! And thank you for coming in to clarify 💯
This reverts commit 26f52c1.
Impierce has tested the plug-in on a range of Android and IOS devices and can confirm great and reliable performance. All our feedback and concerns have been tackled so we have no further feedback to provide. In the next weeks we will be rolling out a much wider public test of our mobile application and will report back any issues if they are encountered. As said in the comment above, we are happy to be mentioned as sponsor but this is non-conditional for us, so we wouldn't complain if Tauri rather removes it or places it on a different location. |
No description provided.