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

Flow swift & FCL IOS maintenance #20

Closed
6 tasks done
lmcmz opened this issue May 13, 2022 · 4 comments
Closed
6 tasks done

Flow swift & FCL IOS maintenance #20

lmcmz opened this issue May 13, 2022 · 4 comments

Comments

@lmcmz
Copy link

lmcmz commented May 13, 2022

flow-swift

Grant category

OpenSource maintenance

Description

There are some new update on flow like secure cadence, dynamic gas fee and HTTP API.
We would like to add those new features into our project.

Problem statement

Currently, some part of the SDK is out of date

Some part can be improved

Proposed solution

  • Update both SDK with latest changes
  • Improve existing function

Impact

  • Developer who wanna build dApp or wallet on Mobile, they can use this latest API in their project.

Milestones and funding

Milestone Deliverables Timeline Risks FLOW requested
1 - HTTP API Flow-swift will support no only gRPC but also HTTP API request ~ 2 weeks Time -
2 - Dynamic Gas Fee By adding a new method in SDK will able to calculate the gas fee dynamically base on compute effect and price effector, before sending it to the chain ~ 2 weeks Need setup a backend with emulator to calculate execution effort -
4 - FCL iOS sign message & account proof Fcl iOS will have the ability to prove authentication ~ 1 weeks Need the wallet provider which will return the proof account serice -
5 - FCL browser window improvement Currently the fcl iOS window will pop as full screen, it should have the ability to pop as page sheet ~ 1 weeks - -
6 - Decode method improving Decoding accessAPI response is hard to use and ugly at the moment, we would like to improve this make it easier to use like fcl js one ~ 1 weeks - -
7 - Maintenance Help and resolve issues for developers who face the issue ~ 4 month - -

Team

Name Role Bio Contact
Hao iOS Engineer The founder of outblock, the core developer of lilico wallet. hao@outblock.io

Milestone completion

  • HTTP API
  • Wallet Connect V2 support
  • FCL iOS user-signed messages & account proof
  • FCL browser window improvement
  • Decode method improving
  • Maintenance
@lmcmz lmcmz mentioned this issue May 13, 2022
@alxflw alxflw added this to New: In review in Grants Overview May 13, 2022
@srinjoyc
Copy link
Contributor

Hi @lmcmz ,

Thanks for the proposal! We have some small questions/suggested changes to the scope of the milestones:

  1. Will you be keeping support for both GRPC and HTTP? It might be lower overhead to completely switch over to HTTP unless there were any technical blockers there.
  2. We can leave out dynamic gas fees for now as there is a lot of complexity being worked on through the FLIP process and the protocol team is still figuring out the mechanics of the system. Once the FCL team gets more information on this, we might be able to add it to this scope, but for now would leave this out.
  3. Milestones 4,5,6 look good!
  4. For "maintenance" the FCL team is planning on releasing a full specification on FCL as it starts to support multiple environments. If possible, we'd like to add "conforming to the new FCL specification" which will include formalizing the methods you support now and ensuring data types and structure are consistent across all implementations.

Lastly, as an action item, would you be able to edit your requested FLOW amounts in USD instead? If you prefer, you can also leave it blank and we can provide suggested amounts. We are going through a small process change and believe it'll be much easier to discuss in terms of USD although the payout will be in FLOW based on a 30-day average token price.

If you're good with all this, leave a reply and we can move forward in the process!

We really appreciate all the great work from you so far 😄

@srinjoyc srinjoyc mentioned this issue May 24, 2022
7 tasks
@lmcmz
Copy link
Author

lmcmz commented May 30, 2022

Hi @srinjoyc

  1. Yes, I was planning to make the as optional to developer which can choose from GRPC or HTTP. But yeah, since there is no blocker with GPC for now, we can lower the priority.
  2. Cool ~
  3. I might wanna add one extra optional feature with Wallet connect v2, so this SDK will support deep link. Let me know how you guys think.
  4. Strong agree with it, I'm happy to keep all SDK aligned.

You guys can provide suggested amount. 😊

@srinjoyc
Copy link
Contributor

srinjoyc commented Jun 1, 2022

Here is what we would suggest for the milestones:

Milestone Deliverables Timeline Risks USD suggested
1 - HTTP API Flow-swift will support the new Access Node HTTP API ~ 2 weeks Time 8,000
2 - Wallet Connect V2 MVP support Deeplinking, Discovery ~ 4 weeks Time, V2 Support, FCL Spec 16,000
4 - FCL iOS sign message & account proof Fcl iOS will have the ability to prove authentication ~ 1 weeks Need the wallet provider which will return the proof account serice 4,000
5 - FCL browser window improvement Currently the fcl iOS window will pop as full screen, it should have the ability to pop as page sheet ~ 1 weeks - 4,000
6 - Decode method improving Decoding accessAPI response is hard to use and ugly at the moment, we would like to improve this make it easier to use like fcl js one ~ 1 weeks - 4,000
7 - Maintenance Resolve bugs, issues and maintain FCL spec compliance ~ 4 month - 16,000

Total USD Suggested: $52,000

Let us know if this looks good and we can have it approved!

@lmcmz
Copy link
Author

lmcmz commented Jun 2, 2022

Challenge accepted🫡

image

@srinjoyc srinjoyc moved this from New: In review to Accepted: In progress in Grants Overview Jun 3, 2022
@alxflw alxflw moved this from Accepted: In progress to Accepted but not active in Grants Overview Jun 13, 2022
@alxflw alxflw added the active label Jun 15, 2022
@alxflw alxflw moved this from Accepted but not active to Accepted: In progress in Grants Overview Jun 15, 2022
@alxflw alxflw added completed and removed active labels Apr 19, 2023
@alxflw alxflw closed this as completed Apr 19, 2023
Grants Overview automation moved this from Accepted: In progress to Closed (withdrawn or passed) Apr 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Grants Overview
Closed (withdrawn or passed)
Development

No branches or pull requests

3 participants