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

Break up GedcomWriter #86

Closed
frizbog opened this issue Sep 27, 2015 · 3 comments
Closed

Break up GedcomWriter #86

frizbog opened this issue Sep 27, 2015 · 3 comments
Assignees
@frizbog
Labels
enhancement
Milestone
Release 3

Comments

@frizbog
Copy link
Owner

frizbog commented Sep 27, 2015 •
edited

GedcomWriter is a magic class, with too much in it. A better way to deal with this would probably be a whole package of classes centered around writing tags and their children, and have GedcomWriter be the root of that activity, rather than doing everything itself.
@frizbog frizbog added this to the Release 3 milestone Sep 27, 2015
@frizbog frizbog mentioned this issue Jul 10, 2016
Closed
@frizbog frizbog self-assigned this Jul 10, 2016
@frizbog
Copy link
Owner Author

frizbog commented Jul 10, 2016

I've started this work in the v3-development branch.

frizbog pushed a commit that referenced this issue Jul 11, 2016
Issue #86 - Splitting up GedcomWriter
d7e8be2
frizbog pushed a commit that referenced this issue Jul 11, 2016
Issue #86 - Splitting up GedcomWriter - Addresses and Submitters
28c37f6
frizbog pushed a commit that referenced this issue Jul 11, 2016
Issue #86 - Splitting up GedcomWriter - Header and Places
15a775f
frizbog pushed a commit that referenced this issue Jul 11, 2016
Issue #86 - Splitting up GedcomWriter - Pretty much the rest of it
32fcb0d
frizbog pushed a commit that referenced this issue Jul 11, 2016
Issue #86 - Splitting up GedcomWriter - final cleanup
cd6f4c0
@frizbog
Copy link
Owner Author

frizbog commented Jul 11, 2016

Completed in v3-development branch

@frizbog
Copy link
Owner Author

frizbog commented Jul 16, 2016

Released in v3.0.0

@frizbog frizbog closed this as completed Jul 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@frizbog frizbog

Labels
enhancement
Projects
None yet
Milestone
Release 3
Development

No branches or pull requests
1 participant
@frizbog