headwire-edge-delivery / eds-self-service-website

0 stars 0 forks source link

Projects with same name #68

Closed icaraps closed 2 weeks ago

icaraps commented 1 month ago

We have an old project named "test" that was created as a drive project that is still live at https://main--test--headwire-self-service.aem.page.

@Lauraa1003 was able to create a DA project named "test" and when previewing content changes with DA, the url https://preview--test.kestrelone.com/ would still proxy https://main--test--headwire-self-service.aem.page which holds the old content from the other "test" drive project.

DriftingSands commented 1 month ago

I think this issue will solve itself down the line. Currently the worker looks for a drive version(headwire-self-service) first, if that is not present it will try DA (da-self-service)

If we further implement DA we can move DA repos off the """temporary""" org and back to headwire-self-service now that it's not locked. There will no longer be two orgs that they are under so an old drive project url would not interfere with a new DA one of the same name.

Or if we don't support DA this is also a non-issue

icaraps commented 1 month ago

If we further implement DA we can move DA repos off the """temporary""" org and back to headwire-self-service now that it's not locked.

I think we should fix it sooner than later because currently, you can create a drive project, delete it, then create a DA project with the same name and your DA site will be broken because it will reference the drive project AFAIU.

DriftingSands commented 3 weeks ago

Waiting on response in the discord... Will try and fix using the EDS API, but we likely need an API key for this

DriftingSands commented 3 weeks ago

Still no response, but I had an idea that I quickly tested and it worked.

I can implement a fix for this, but it will a make the deletion request take a lot longer. We will be sending one deletion request per page. If I ever get a response there might be a way of doing it in one request

DriftingSands commented 2 weeks ago

Workaround added about 1-2 seconds to speed index of drive projects. So it was reverted.

We have two options now:

DriftingSands commented 2 weeks ago

Update frontend, backend & worker to use da-- prefix