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

Thoughts on v2 #47

Open
1 of 7 tasks
emyann opened this issue Feb 9, 2019 · 10 comments
Open
1 of 7 tasks

Thoughts on v2 #47

emyann opened this issue Feb 9, 2019 · 10 comments
Assignees
Projects

Comments

@emyann
Copy link
Member

emyann commented Feb 9, 2019

Planning a v2.0.0 for Morphism with those changes:

Please share any ideas or suggestions you would find valuable for a next version :)

@emyann emyann self-assigned this Feb 9, 2019
@emyann emyann added this to To do in Morphism via automation Feb 9, 2019
@emyann emyann changed the title Drop registry support in v2 Thoughts on v2 Feb 9, 2019
@emyann emyann pinned this issue May 7, 2019
@davej
Copy link

davej commented May 25, 2019

Awesome, this is a really nice lib.

Probably way outside the scope of what this library is for but it would be cool to be able to output the schema transformations on some sort of diagram. Perhaps an online tool or CLI that outputs to something like d3.js.

@emyann
Copy link
Member Author

emyann commented May 26, 2019

@davej This is definitely something I had thought about! I really love the idea, I just didn't know if this could be valuable. Thank you a lot for bringing this on the table, I'll add it to the list :)

@reitsma
Copy link

reitsma commented Jun 12, 2019

This looks really nice. I am using the JOLT java library in the backend. For the frontend transformations I consider Morphism. I have one specific requirement: The transformation schema must be expressed in JSON. As far as I understand, that can be done with Morphism, my own code will do the (trivial) parse of the JSON before handing the scheme to Morphism.

@emyann
Copy link
Member Author

emyann commented Jun 12, 2019

@reitsma Thank you for considering using Morphism! I don't see any actual blocker for your use case ? If you're able to parse and revive the functions you might have in your schema, it's going to work straight away.

@cjancsar
Copy link

cjancsar commented Oct 9, 2019

Might be nice to have easy hooks to sanitize data during mapping. Can currently do this manually with a function mapper, but might be nice to have a more abstract way.

We are not using typescript at the moment unfortunately.

Think of an integration with sanitizers like validator.js, like express-validator does.

We are using this lib to connect backend services with our internal micro-services, as an ETL-like process. This library is great to map the data, but needs a bit more in the transformation-side.

We are just starting to use it, and I would be happy to help out in the coming months.

@emyann
Copy link
Member Author

emyann commented Oct 9, 2019

@cjancsar This is awesome that you bring this on the table as I'm actually working on the Validation side of Morphism! This is only available on the next branch of this library (npm install morphism@next).

This is not yet ready for production but you can see a sneak peek here:

I added some basic string validators here https://github.com/nobrainr/morphism/blob/next/src/validation/validators/StringValidator.ts like min, max, length, alphanum, regex and planning to add more in the future.

Is it similar to what you had in mind ?

@cjancsar
Copy link

cjancsar commented Oct 9, 2019

@emyann yes! That is exactly what I was thinking.

Do the validators have the ability to change the data, or are they read-only, and only throw errors?

Are you re-creating all your own validators? I know you likely don't want to add an external dependency, but the lib might also get more recognition and wide-spread use if it connected to the already-awesome validator.js which has tons of existing validators.

It also has sanitizers, which coerce the incoming value to another format / clean the data. Would also be nice to see that.

For example:

import { morphism, createSchema, Validation, Sanitization, reporter } from 'morphism';

const source = { name: 'Cassandra', id: '14' };
const schema = createSchema({ 
	id: { 
		fn: (source:any) => source.id, 
		validation: Validation.isInt(),
                sanitization: Sanitization.toInt()
	} 
});

> output = { name: 'Cassandra', id: 14 }

Pretty basic example, but you get the point.

@emyann
Copy link
Member Author

emyann commented Oct 9, 2019

Awesome!

Do the validators have the ability to change the data, or are they read-only, and only throw errors?

Yes actually it does coerce the value to another format and I wanted to give the user the ability to choose the behavior, but I really love the suggestions you shared (Sanitization, integration with validator.js). I'll work on making the validation interface more composable so that it would be easy to plug any other validation / sanitization library rather than taking a dependency on a specific library (if some prefer joi over validator.js it would be possible)

@cjancsar
Copy link

cjancsar commented Oct 9, 2019

@emyann cool! Let me know if there are any low-hanging requirements and I will try and spend some time helping you out.

@emyann
Copy link
Member Author

emyann commented Oct 9, 2019

@cjancsar Yes of course, thank you for offering your help! I'll wrap my head around the integration with 3rd party libraries and prepare the ground for the work. I'll get back to you whenever I start this refactoring.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Morphism
  
To do
Development

No branches or pull requests

4 participants