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.

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

Merged
merged 49 commits into from
Sep 14, 2023
Merged

Conversation

lucasfernog
Copy link
Member

No description provided.

@socket-security
Copy link

socket-security bot commented Aug 7, 2023

No top level dependency changes detected. Learn more about Socket for GitHub ↗︎

@lucasfernog lucasfernog marked this pull request as ready for review September 7, 2023 03:00
amrbashir
amrbashir previously approved these changes Sep 7, 2023
Copy link
Member

@amrbashir amrbashir left a 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");
Copy link
Member

Choose a reason for hiding this comment

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

local path

Copy link
Member

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

Copy link
Member

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.

Comment on lines 75 to 92
## 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>
Copy link
Member

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.

Copy link
Member

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.

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 ;)

Copy link
Member

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 💯

@JelleMillenaar
Copy link

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.

@lucasfernog lucasfernog merged commit 454428c into tauri-apps:v2 Sep 14, 2023
@lucasfernog lucasfernog deleted the feat/scanner branch September 14, 2023 12:34
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.

5 participants