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

api: add dc_replace_webxdc() #4740

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

link2xt
Copy link
Collaborator

@link2xt link2xt commented Sep 20, 2023

This is an alternative to #4532.

The idea is to add a new JavaScript API window.webxdc.replaceWebxdc allowing the WebXDC app to replace itself. This will allow the apps to upgrade themselves by requesting a base64-encoded blob from a bot.

This will also allow the apps to modify themselves by repackaging all the files with necessary modifications (e.g. updated configuration). E.g. a TiddlyWiki port may replace itself in-place instead of saving a new copy of self to chat each time.

@link2xt link2xt force-pushed the link2xt/replace-webxdc-api branch 2 times, most recently from 1ce943a to 6976e9b Compare September 21, 2023 14:17
@link2xt link2xt marked this pull request as ready for review September 21, 2023 15:31
@link2xt link2xt force-pushed the link2xt/replace-webxdc-api branch 2 times, most recently from 438e2db to 3549fcb Compare September 22, 2023 10:35
@link2xt
Copy link
Collaborator Author

link2xt commented Sep 22, 2023

It is actually ready, but needs a JSON-RPC API as well.

Copy link
Collaborator

@iequidoo iequidoo left a comment

Choose a reason for hiding this comment

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

It is actually ready, but needs a JSON-RPC API as well.

Looks like this doesn't prevent merging though

@link2xt
Copy link
Collaborator Author

link2xt commented Sep 30, 2023

The reason it is in the draft mode currently is because there is no decision on webxdc/website#61

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