humhub / rest

HumHub Rest API Module
24 stars 24 forks source link

Add content scheduling to content #121

Closed gevorgmansuryan closed 1 year ago

gevorgmansuryan commented 1 year ago

https://github.com/humhub/rest/issues/119

luke- commented 1 year ago

@gevorgmansuryan Is this ready? Seems the tests are failing. https://github.com/humhub/rest/actions

gevorgmansuryan commented 1 year ago

@luke- yes, this is ready. But seems tests are failing in all branches. I've created test empty branch and tests are failing. Seems somethings wrong with datepicker. I'll take a look

yurabakhtin commented 1 year ago

@gevorgmansuryan Is this ready? Seems the tests are failing. https://github.com/humhub/rest/actions

@luke- @gevorgmansuryan Error in the test AuthCest: Login by Basic auth and JWT bearer token is not related to this PR, I tried to fix it too here https://github.com/humhub/humhub-internal/issues/191, but still don't have idea why it is broken only on develop and on the github side.

yurabakhtin commented 1 year ago

@luke- @gevorgmansuryan When all changes will be done related to state and scheduled_at columns we should also update doc/swagger files in the modules where REST API is used, like calendar, wiki, tasks and etc.

gevorgmansuryan commented 1 year ago

@gevorgmansuryan Is this ready? Seems the tests are failing. https://github.com/humhub/rest/actions

@luke- @gevorgmansuryan Error in the test AuthCest: Login by Basic auth and JWT bearer token is not related to this PR, I tried to fix it too here humhub/humhub-internal#191 (comment), but still don't have idea why it is broken only on develop and on the github side.

@luke- @yurabakhtin On my local env it's failing too, after composer update. Package firebase/php-jwt was updated. Fixed

luke- commented 1 year ago

@luke- @gevorgmansuryan When all changes will be done related to state and scheduled_at columns we should also update doc/swagger files in the modules where REST API is used, like calendar, wiki, tasks and etc.

@yurabakhtin Good point.

@gevorgmansuryan Can you please also add, following attributes to the REST API

After merge this PR, can you please rebuild the docs/swagger files for the modules.

Thanks!