plone.restapi icon indicating copy to clipboard operation
plone.restapi copied to clipboard

@aliases Endpoint

Open Petchesi-Iulian opened this issue 3 years ago • 23 comments

For #1393

Petchesi-Iulian avatar May 04 '22 10:05 Petchesi-Iulian

@Petchesi-Iulian thanks for creating this Pull Request and helping to improve Plone!

TL;DR: Finish pushing changes, pass all other checks, then paste a comment:

@jenkins-plone-org please run jobs

To ensure that these changes do not break other parts of Plone, the Plone test suite matrix needs to pass, but it takes 30-60 min. Other CI checks are usually much faster and the Plone Jenkins resources are limited, so when done pushing changes and all other checks pass either start all Jenkins PR jobs yourself, or simply add the comment above in this PR to start all the jobs automatically.

Happy hacking!

mister-roboto avatar May 04 '22 10:05 mister-roboto

Deploy Preview for plone-restapi ready!

Name Link
Latest commit 485184d02a58730ef4c829416a85438a126b30a3
Latest deploy log https://app.netlify.com/sites/plone-restapi/deploys/63163929bf3c410009da06e7
Deploy Preview https://deploy-preview-1398--plone-restapi.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

netlify[bot] avatar May 04 '22 10:05 netlify[bot]

@jenkins-plone-org please run jobs

Petchesi-Iulian avatar May 05 '22 12:05 Petchesi-Iulian

@jenkins-plone-org please run jobs

Petchesi-Iulian avatar May 05 '22 13:05 Petchesi-Iulian

@jenkins-plone-org please run jobs

Petchesi-Iulian avatar May 06 '22 10:05 Petchesi-Iulian

@jenkins-plone-org please run jobs

Petchesi-Iulian avatar May 06 '22 12:05 Petchesi-Iulian

@jenkins-plone-org please run jobs

Petchesi-Iulian avatar May 06 '22 13:05 Petchesi-Iulian

@jenkins-plone-org please run jobs

Petchesi-Iulian avatar May 07 '22 08:05 Petchesi-Iulian

@jenkins-plone-org please run jobs

Petchesi-Iulian avatar May 09 '22 10:05 Petchesi-Iulian

@jenkins-plone-org please run jobs

avoinea avatar May 09 '22 13:05 avoinea

@jenkins-plone-org please run jobs

Petchesi-Iulian avatar May 24 '22 13:05 Petchesi-Iulian

@jenkins-plone-org please run jobs

Petchesi-Iulian avatar May 25 '22 14:05 Petchesi-Iulian

@jenkins-plone-org please run jobs

Petchesi-Iulian avatar May 25 '22 14:05 Petchesi-Iulian

@jenkins-plone-org please run jobs

Petchesi-Iulian avatar May 31 '22 14:05 Petchesi-Iulian

@jenkins-plone-org please run jobs

fredvd avatar Jun 07 '22 11:06 fredvd

@fredvd Are the failing jenkins test still not-related to this PR or it's a real issue?

avoinea avatar Jun 14 '22 13:06 avoinea

@jenkins-plone-org please run jobs

avoinea avatar Jun 14 '22 13:06 avoinea

Are the failing jenkins test still not-related to this PR or it's a real issue?

@avoinea These are not related to this PR, there are still a few flakey robot tests left on plone.schema editor and the generic Fields robot tests that fail occasionally. Those are also failing in other tests.

I was fumbling around a bit to check the failures/success for this specific PR on the Pull request jobs on jenkins, just learned you can paste the PR URL in the filter on the PR job page to see them:

pr filter on url

fredvd avatar Jun 14 '22 14:06 fredvd

@avoinea The robot tests are now zero, but other 'normal' tests fail.

https://jenkins.plone.org/job/pull-request-6.0-3.8/1499/

fredvd avatar Jun 15 '22 08:06 fredvd

@jenkins-plone-org please run jobs

Petchesi-Iulian avatar Jun 16 '22 12:06 Petchesi-Iulian

@jenkins-plone-org please run jobs

Petchesi-Iulian avatar Jun 30 '22 15:06 Petchesi-Iulian

@jenkins-plone-org please run jobs

Petchesi-Iulian avatar Aug 16 '22 11:08 Petchesi-Iulian

@jenkins-plone-org please run jobs

avoinea avatar Aug 30 '22 08:08 avoinea

@Petchesi-Iulian @avoinea just wondering. The current @aliases endpoint does not allow filtering by aliases that have been added manually or automatically, is that correct?

tisto avatar Jun 14 '24 13:06 tisto

@tisto Yes, I think you're right. There is no way to filter them by the manual criteria :thinking:

avoinea avatar Jun 14 '24 13:06 avoinea

@avoinea thanks for the feedback! I created an issue for this:

https://github.com/plone/plone.restapi/issues/1791

I also noticed that we do not have any way to filter for a date range. I also created a ticket for this:

https://github.com/plone/plone.restapi/issues/1792

We should also add the functionality to upload/download aliases as CSV file:

https://github.com/plone/plone.restapi/issues/1793

Once we have the REST API endpoints in place, we could enhance the Volto URL control panel to expose this new functionality.

Are you folks interested in those functionalities? We (at kitconcept) are currently working with the ClassicUI controlpanel but this is not a long-term solution. Therefore I'd love to see if we would catch up with Volto.

tisto avatar Jun 15 '24 05:06 tisto