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

Flux command stream-to-xml should also be available as encode-xml #308

Closed
fsteeg opened this issue Sep 24, 2019 · 2 comments
Closed

Flux command stream-to-xml should also be available as encode-xml #308

fsteeg opened this issue Sep 24, 2019 · 2 comments
Assignees
@dr0i

Comments

@fsteeg
Copy link
Member

fsteeg commented Sep 24, 2019

For consistent naming of Flux commands, stream-to-xml should also be available as encode-xml.

Would this work as an additional command, to keep compatible with existing scripts?
@dr0i
Copy link
Member

dr0i commented Sep 24, 2019

+1

@dr0i dr0i self-assigned this Sep 24, 2019
dr0i added a commit to hbz/metafacture-core that referenced this issue Sep 30, 2019
@dr0i
Add another name to flux command "stream-to-xml"
e21c24c 
This name points to SimpleXmlEncoder.
Keeping the old name for compatibility reasons.

See metafacture#308.
@dr0i dr0i mentioned this issue Sep 30, 2019
Merged
@dr0i
Copy link
Member

dr0i commented Oct 24, 2019

Merged into master. Closing.

@dr0i dr0i closed this as completed Oct 24, 2019
blackwinter added a commit that referenced this issue Dec 13, 2024
@blackwinter
Merge pull request #308 from metafacture/116-destructiveAddField
f2e6472 
Change `add_field()` Fix function to destructive behaviour for Catmandu compatibility.
blackwinter pushed a commit that referenced this issue Dec 13, 2024
@TobiasNx
Update README.md
511e9ef 
Adjust the documentation to the actual behaviour #308
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dr0i dr0i

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@fsteeg @dr0i