DataSync 2.0

[wtrocki]

Purpose of this branch is to bring latest and greatest features from Graphback and Offix.

• Include early version of Graphback CRUD (https://graphqlcrud.org)
• Include pagination support
• Include Offix DataStore
• Include advanced forms and relationships between models.

[wtrocki]

OVP showed us some problems with general user experience:

• Scalars are very important everywhere. Not supporting scalars is big deal.

• Annotations are hard to work (as they require reference) Our docs do not have reference for those (they are spread across different topics and hard to find

• Relationships and types are not always easy to build

• Database specific ID's etc not always work.

That is why I'm going to use this as a long-living branch to bring full features of graphback and measure impact on offix

[wtrocki]

@kingsleyzissou Quick opinion. Do we want to have TaskView screen that will allow add comments. I have done some preparation for this,

[kingsleyzissou]

@wtrocki, I think if it's not too much work and it's easy to maintain then we can go ahead with a TaskView

[kingsleyzissou]

@wtrocki the root query should be holding a reference to the normalised cache objects, is that correct?

And here it just looks like it’s duplicating them in both places. Have I understood correctly?

[wtrocki]

Requires custom offix: aerogear/offix#504

[wtrocki]

I have updated offix to make this work.

@kingsleyzissou would you have time to play with this branch in context of the: https://github.com/aerogear/offix/issues/499

Some things might require some quick fixes still (I do not have much time to get that done now) but it looks we can somehow patch offix to work with latest graphback and unblock release while still keeping the work on the datastore.

Any feedback is greatly appreciated.

What is outstanding here:

• enabling read and updates (there are wrong cache operations there
• figuring out comment updates (requires custom update function - we cannot generate one for this as it will be too complex - currently works by refetch queries)
• Rendering of the comment component should use a different form. Comment is read only and this component do not seems to be supporting it well.

[kingsleyzissou]

Okay leave the outstanding stuff, I will have a look at that and try get that sorted.

Just busy testing the updates on the sample app and then I'll come around to this.

[kingsleyzissou]

https://github.com/aerogear/datasync-starter/issues/435

[kingsleyzissou]

https://github.com/aerogear/datasync-starter/issues/436

[wtrocki]

I think it will be easier for us to merge this to master with alpha release after we know that keycloak works (@kingsleyzissou since you have been doing keycloak before can you try this with keycloak as well).

If we merge then we can add incremental changes and proper graphback release once it happens. In the meantime this will close offix 1.5 work and we could fully leave this repository for the moment till we have some alpha releases of Offix 2.0.

@kingsleyzissou Does this make sense?

Ping me for quick review of the client once you are happy with this :D

[kingsleyzissou]

@wtrocki sure I will review the keycloak stuff. I will try find a quick win for the optimistic responses, I think the Offix 2.0 stuff should take priority.

Will ping you as soon as it's ready

[kingsleyzissou]

@wtrocki I have updated those small changes.

Doesn't look like the keycloak role stuff is working properly. But will try debug that on Monday morning.

Have a good weekend.

[wtrocki]

This is ready to be merged. However some clarification and verification of the walkthrough will be needed. CC @mmusil