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

Keep use of ModelMarshaler only where really needed #286

Open
otsakir opened this issue Jan 15, 2018 · 1 comment
Open

Keep use of ModelMarshaler only where really needed #286

otsakir opened this issue Jan 15, 2018 · 1 comment
Labels
Refactoring / Tech debt

Comments

@otsakir
Copy link
Contributor

otsakir commented Jan 15, 2018

ModelMarshaler is a component converts data models to json and vice versa. It has some type adapters created by default for handling some special RVD models in a special way. However it's not needed in all places.

We should remove the dependency to ModelMarshaler in *RestServices. It will reduce coupling a little more.
@otsakir
Copy link
Contributor Author

otsakir commented Feb 14, 2018

This also relates to #35 which contains that involved refactoring marshaller class.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Refactoring / Tech debt
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@otsakir