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

Feature/add telegram #75

Draft
wants to merge 37 commits into
base: main
Choose a base branch
from
Draft

Feature/add telegram #75

wants to merge 37 commits into from

Conversation

Arth-y
Copy link
Contributor

@Arth-y Arth-y commented Jun 6, 2024

⚠️ To have it work on your machine, you need to clone https://github.com/iExecBlockchainComputing/web3telegram
Look at how it is referenced in package.json:

"@iexec/web3telegram": "file:../web3telegram-sdk",

and to run npm run build inside web3telegram. A /dist folder will be generated.


⚠️ web3telegram iApp is not yet published to prod (missing Intel private key signing)
Hence can only run on debug v8 learn workerpool?
Meaning that it can only access protected data with their secret in the debug SMS?
→ Code as it is in this PR probably needs some updates to work.


Add Telegram feature to the use-case-demo :

  • New ProtectedData type : telegram
  • Add "Send Telegram" to navbar to send a message using a telegram bot
Capture d’écran 2024-12-20 à 15 45 12 Capture d’écran 2024-12-20 à 15 30 39 Capture d’écran 2024-12-20 à 15 35 09 Capture d’écran 2024-12-20 à 15 36 29

image

TODO: Rename this repo?

@Arth-y Arth-y self-assigned this Jun 6, 2024
@Arth-y Arth-y requested a review from cedric25 June 6, 2024 13:38
Copy link
Contributor

@cedric25 cedric25 left a comment

Choose a reason for hiding this comment

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

Petite passe de review pour essayer de se rapprocher le plus possible d'une mise en production.

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.

2 participants