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.

Sign up for GitHub

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

Jump to bottom

MCSnapshotBrowser should be ported to Spec #17399

Open
Ducasse opened this issue Nov 17, 2024 · 1 comment · May be fixed by #17468
Open

MCSnapshotBrowser should be ported to Spec #17399

Ducasse opened this issue Nov 17, 2024 · 1 comment · May be fixed by #17468
Assignees
@hernanmd
Labels
Project: Prepare morphic phase out Type: Cleanup Type: UX
Milestone
13.0.0

Comments

@Ducasse
Copy link
Member

Ducasse commented Nov 17, 2024

Right now the snapshot browser is implemented in Morphic.
It should be redone with Spec.
We can access it from the browse in monticello.

I though that cyril cleaned it. @jecisc tell us more.

Capture 2024-11-17 at 18 05 47
@jecisc
Copy link
Member

jecisc commented Nov 18, 2024

For this one I'm not sure what to do.

We have Calypso that can be open on a Ring model. So I was thinking about casting a Monticello model into a ring one and use Calypso to browse the code.

But we have no test on Calypso with Ring so it breaks a lot. I don't know what is the best between redoing the Snapshot browser in Spec on using Calypso to browse the code.

@hernanmd hernanmd self-assigned this Nov 26, 2024
@hernanmd hernanmd added this to the 13.0.0 milestone Nov 26, 2024
@hernanmd hernanmd linked a pull request Nov 28, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hernanmd hernanmd

Labels
Project: Prepare morphic phase out Type: Cleanup Type: UX
Projects
None yet
Milestone
13.0.0
Development

Successfully merging a pull request may close this issue.

Ported Monticello Snapshot Browser to Spec 2 hernanmd/pharo
3 participants
@Ducasse @hernanmd @jecisc