berlindb / core

All of the required core code
MIT License
252 stars 27 forks source link

feat(rest): first implementation #150

Open Mte90 opened 2 years ago

Mte90 commented 2 years ago

Ref: https://github.com/berlindb/core/issues/26

For an example to test it https://github.com/Mte90/berlindb-rest/

I will update the example when will be merged https://github.com/berlindb/wordpress-example

Mte90 commented 2 years ago

I was thinking too that some redefine like for the table etc wasn't necessary but this library just extend various classes and they are not connected each other so those data are not shared. So in the meantime the patch let everything to work and those changes require some internals changes in BerlinDB that I think it's not the case to discuss here (to keep the focus on the REST stuff, maybe open a ticket for that?).

During the development of this integration I saw this issue in that PR:

Mte90 commented 1 year ago

I was forgetting a little things, I just configured the doc comments for 3.0.0 release.

I still waiting for a review as I am curious for feedback :-D

Mte90 commented 1 year ago

In the meantime I am using my fork in a plugin I am working on, so I have another test field.

JJJ commented 1 year ago

Haven't forgotten about this. 👋

Just been busy working on other 3.0 changes to hopefully make your work easier to merge/adopt. 🍕

Mte90 commented 1 year ago

Thanks I just used BerlinDB in this period #23 as per that ticket and I think that the example need improvements until the documentation is not out there.

Mte90 commented 1 year ago

just a ping

mircobabini commented 1 year ago

Can't wait to see this merged too. If you need some help to run tests or everything ✋🏻✋🏻✋🏻

Mte90 commented 1 year ago

ping