-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[Tracking / LOW] [Wave Collect] [HybridApp] Investigate Native Share for HybridApp #36613
Comments
Hey @grgia, I'm from Software Mansion and I'll be handling this issue |
📣 @BrtqKr! 📣
|
@BrtqKr could you post a quick update to this issue for tracking? |
The screens for this feature are mostly done, we're connecting them with onyx. Right now we're working on reusing the existing logic from the native side and expanding it with the share extension serving as the "router", which would point to the proper OldDot/NewDot flow without affecting the old logic. We're still waiting for the ios setup for AppGroups, which is already done for the old dot and is required if we want to keep everything consistent. |
See https://expensify.slack.com/archives/C04878MDF34/p1710412834178559 |
I need to review the doc for this one. Could you link here as well @BrtqKr ? I want to unblock this this week :) |
Took a look at this today @BrtqKr. I left a comment in the doc, but I think we can use a lot of the HL from the previous doc. Do you feel comfortable running with that and moving over the information that applies to your design? If necessary, we can also get new mock ups specific to hybrid app, we just need to center on what we haven't covered if that's the case |
Was deprioritized, but we should get the HL updated and get this wrapped up in the next release cycle |
Still making steady progress |
IOS and Android are both in progress |
Same as above, draft PRs expected for this week |
Android and IOS are in progress, still waiting on PRs |
Same as above |
Making monthly for holidays |
Chatting about unblocking IOS build here - https://expensify.slack.com/archives/C01GTK53T8Q/p1736339255068649 |
PR nearing final review! |
in progress, still held on provisioning issue |
@grgia assigned myself as BZ in case you ever need help (and... to follow along) |
Working on the provisioning profile locally. You can find the convo here: https://expensify.slack.com/archives/C03TQ48KC/p1738889505601349 |
Background:
V0 (NewDot-only design) tracking issue / design doc.
Moving Forward:
We originally designed this new feature for NewDot only, but as we move towards one single app (Hybrid App) on the Old Expensify App, we need to leverage/extend the existing share-to-scan receipt feature to support common chat-based sharing.
This means that in addition to Share to Scan receipt, a user will be able to share attachments/images/text/links to chats in New Expensify.
We may need to HOLD this project as HybridApp plans change or solidify, but for now we can use this issue to share results from investigations
The text was updated successfully, but these errors were encountered: