Stand Up 6/21
Date
Jun 21, 2022
Note Taker
Property 1
Deprecated URL:
- when testing workflow, checkout URL no longer works, and there was no documentation (there are different ways to add URL, two new parameters are needed - create sales channel discount (make a request to Shopify with product idea and variant and they respond with the proper checkout URL, it needs to be on server side (rose)) - (function in banklesscheckout.js in assets folder) — roseja (load success screen function same, just update the URL)
- right now, reroutes people back to cart ^
- multisig:
- nft branch: done, not merged yet while trying to fix checkout, but could be solved separately
- URL: GraphQL limitation to create the checkout, the GQL works, but the authentication doesn’t worked, checkout url we have to use the storefront one, which gives an error (error: client mutate not function). Should work as manually it works, but the application authenticating with the endpoint is problematic.
prad:
Next Week
- production app by weekend
- might help roseja with url debugging
Next Week
roseja:
- debugging URL issue
Wallet:
- prad knowledge sessions: a place to send funds
- check parcel wait list
- or polygon gnosis
Prad:
There was a delay on Shopify, which made it hard to test, there was a glitch, and the code would function differently on the dev store and the published version. There was a 2-3 hour bug.
Roseja:
Errors locally and the order wasn’t being created
webhook working ok on the dev store, on test store and there is a 403 error
It is not a scope issue
Not sure about the 403
Prad recommendation: reinstall
Webhook:
- the reason with 85% confidence: session storage part needs to be fixed, causing webhook issues >> using redis, heroku redis is free so we can use that for now.
- everything is connected to dev server will be moved to a different production one once the server is setup
- webhook needs response; we do respond when it is enable and was working for roseja, not sure how to recreate that again through the app code >> potentially refreshing the session and see if that will create that in this way, or setting up a time to do it
Team editor:
does roseja need a place for prad to log his changes
Security: still an issue
Rotate api keys
two-step authentication
Polishing the approval process:
- in bankless: they were requiring two approvals for each pr, or just one approval
- whoever works on the feature merges the feature
Agenda + notes
- Updates:
- Prad: -1 week
- last week
- no buying with erc20 tokens for redeemable products, tested
- found URL issue and looked into
- some updates on spec sheet
- polygon grant: deck (mostly description of the product/client/tech/some suggested monetizaion model (transaction fee), in the future perhaps subscriptions on roadmap)
- filter for cart totally empty for NFT redemption, for token it will create transaction of zero tokens )
- button in products placement and fixing issues it was created
- Got the Bankless-Checkout embedded on product details page.
- research and documentation (3 lines of comments to clarify)
- would move to use multiple files for the code (split to 3 - 4 files) and refactoring functions — code is looking clearer and easier to understand
- spec sheet >> in progress
- talk to BA and polygon dev advocate for polygon - in process
- BA person connected to the BA grant writer
- Next week:
- use fathom for analytics
- generic API for different tokens >> on hold
- production app up and running >> in progress
- add error messages for payment with erc20, when cart is empty >> low priority, on hold after
- prad share screen w Jen for transaction failed
- Challenge:
- Questions:
- roseja: -
- Did the checkout button changes, not sure about deploying (prad tests on dev store, then we publish and we check if it working on usm store after publishing) - you can merge of develop as long as there are no conflicts
- changed notification - won’t show any checkout buttons
- figured out the reason webhook is failing sporadically, perhaps
- infosec meeting:
- we don’t have to store an email or personal identifiable information
- they mentioned for shopify admins, everyone should have two-factor authentication for the actual shopify store - groggy can enforce it as an owner, fibo get two factor auth
- GDPR, privacy issues of storing customer data, not relevant because we are not storing PII atm
- Next week:
- deprecated URL issue (bug, priority high)
- maybe webhook session issue
- talk to bog about security for the data
- looking into 403 error for order create webhook, which could help with some of the issues we’re having
- adding the bankless checkout button for the pop up when add to cart
- auth-token and how it can help with the webhook issue
- Challenge:
Last week:
Jenetics:
- transaction failed issue >>
improving dev workflow, working on bugs
Suggested features:
1- revenue split
2- supporting eth mainnet
3- nft receipts
4- token gating