Skip to content
This repository has been archived by the owner on Nov 21, 2024. It is now read-only.

add workflows to admin #12

Merged
merged 1 commit into from
Nov 14, 2023
Merged

add workflows to admin #12

merged 1 commit into from
Nov 14, 2023

Conversation

MJedr
Copy link
Contributor

@MJedr MJedr commented Nov 10, 2023

No description provided.

@MJedr MJedr force-pushed the add-workflows-to-admin branch from 348a7ba to f5f6879 Compare November 13, 2023 12:48
@MJedr MJedr force-pushed the add-workflows-to-admin branch from f5f6879 to 514f1da Compare November 13, 2023 12:52
from django.urls import include, path

urlpatterns = [
path("workflows/", include("workflows.urls")),
path("admin/", admin.site.urls),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we keep both?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yep, but not sure if we need both - do we need extra admin url on backoffice.inspire.net/workflows/backoffice?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean, backoffice.inspire.net/dashboard and backoffice.inspire.net/admin

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, so it's defined in config.urls, this adds url backoffice.net/workflows/admin which we don't need

@drjova drjova merged commit 3aacdd2 into main Nov 14, 2023
@drjova drjova deleted the add-workflows-to-admin branch November 14, 2023 09:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants