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

Make published_to_js (now publish_to_js) official API through AbstractPlutoDingetjes.jl (v2) #2608

Merged
merged 13 commits into from
Jul 20, 2023

Conversation

disberd
Copy link
Contributor

@disberd disberd commented Jul 16, 2023

This resurrects #2162 and adds some front-end tests.

Second tests is currently commented out as it's broken due to #2092, but @fonsp had a very productive afternoon in Graz fixing that out in #2607

Works in combination with JuliaPluto/AbstractPlutoDingetjes.jl#11

@github-actions
Copy link
Contributor

Try this Pull Request!

Open Julia and type:

julia> import Pkg
julia> Pkg.activate(temp=true)
julia> Pkg.add(url="https://github.com/disberd/Pluto.jl", rev="published_to_js")
julia> using Pluto

@disberd disberd changed the title move publis_to_js to AbstractPlutoDingetjes (v2) move publish_to_js to AbstractPlutoDingetjes (v2) Jul 16, 2023
@fonsp fonsp changed the title move publish_to_js to AbstractPlutoDingetjes (v2) Make publish_to_js (now published_to_js) official API through AbstractPlutoDingetjes.jl (v2) Jul 17, 2023
@fonsp fonsp changed the title Make publish_to_js (now published_to_js) official API through AbstractPlutoDingetjes.jl (v2) Make published_to_js (now publish_to_js) official API through AbstractPlutoDingetjes.jl (v2) Jul 17, 2023
@fonsp
Copy link
Owner

fonsp commented Jul 17, 2023

Still missing is an assertpackable call :) Will do that sometime soon

@fonsp
Copy link
Owner

fonsp commented Jul 17, 2023

I think I broke embed_display in #2607 so lets fix that here

@disberd
Copy link
Contributor Author

disberd commented Jul 18, 2023

What is broken in embed_display?
I tried a simple call in the output and in the logs but I can't find problems:
image

@fonsp
Copy link
Owner

fonsp commented Jul 20, 2023

Yep looks like it still works!

@fonsp fonsp merged commit 1d2c303 into fonsp:main Jul 20, 2023
15 of 16 checks passed
@fonsp fonsp added backend Concerning the julia server and runtime display & PlutoRunner & AbstractPlutoDingetjes.jl labels Sep 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Concerning the julia server and runtime display & PlutoRunner & AbstractPlutoDingetjes.jl
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants