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

Us143 Export Document as serialized object #168

Merged
merged 8 commits into from
Aug 7, 2020
Merged

Conversation

konradh
Copy link
Contributor

@konradh konradh commented Jul 27, 2020

No description provided.

@konradh konradh changed the base branch from dev to US88 July 27, 2020 10:50
@konradh konradh changed the base branch from US88 to US88-draft July 27, 2020 10:50
@konradh konradh linked an issue Jul 27, 2020 that may be closed by this pull request
3 tasks
@frcroth
Copy link
Contributor

frcroth commented Jul 30, 2020

Please add tests

@sirkrypt0
Copy link
Contributor

Intended to merge into US88?

@frcroth frcroth changed the base branch from US88-draft to dev August 6, 2020 13:40
@sirkrypt0
Copy link
Contributor

Why is it no strategy pattern like with export?

@frcroth
Copy link
Contributor

frcroth commented Aug 6, 2020

Why is it no strategy pattern like with export?

I guess you could argue with XP / Triangulation. As we only implement one Import Strategy a full pattern would go against these patterns

@konradh konradh changed the base branch from dev to fixed-ui-element-height August 7, 2020 08:12
@konradh konradh merged commit d04e998 into fixed-ui-element-height Aug 7, 2020
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.

Export as Squeak Document
3 participants