dyne / zenpub

Reflow federated economic network
GNU Affero General Public License v3.0
13 stars 3 forks source link

Assess ZenPub and Bonfire backwards GraphQL Compatibility #69

Open pral2a opened 3 years ago

pral2a commented 3 years ago

We detected the following GraphQL models (queries or mutations) originally available in ZenPub are not available in Bonfire (using Dyne staging instance at http://135.181.35.156:4000/api/explore)

We documented each issue for the Bonfire team to use the comments to detail how to proceed: remove functionality, update interface, wait for a solution, etc.

Registration and Authentication

Users log in, sign up, password recovery Requests authentication (docs missing ?) https://github.com/reflow-project/weloop/issues/136

User(s) and profile

User reflow-project/weloop#135 needs to be map to Agent reflow-project/weloop#124 Me reflow-project/weloop#131

Social network features in use

Activity reflow-project/weloop#123 (maybe not critical unless it also provides information about the value flows activity) Community reflow-project/weloop#129 Comment reflow-project/weloop#128 Thread reflow-project/weloop#126 Like reflow-project/weloop#134 Flag reflow-project/weloop#132

Temporary deprecated social network features

Collection reflow-project/weloop#127 Resource reflow-project/weloop#130

Not in use

~Instance reflow-project/weloop#133~ ~WebMetadata reflow-project/weloop#125~

sbocconi commented 2 years ago

Hello @pral2a,

I think there is a mix up in referencing the github issues in the text above. For example,

User reflow-project/weloop#123 needs to be map to Agent reflow-project/weloop#134 Me reflow-project/weloop#127

should be (I think):

User reflow-project/weloop#135 needs to be map to Agent reflow-project/weloop#124 Me reflow-project/weloop#131

Also in other parts of the text the issues are swapped, see for example:

Like reflow-project/weloop#124

pral2a commented 2 years ago

Hello @sbocconi

Thank you very much for the comment. Indeed, there were multiple mistakes in the issue references.

@adam-burns Now they are fully fixed

Hope it helps!