-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: delete Org, Project and Manage Branches #39
feat: delete Org, Project and Manage Branches #39
Conversation
@@ -12,13 +12,17 @@ | |||
<meta name="msapplication-TileColor" content="#da532c"> | |||
<meta name="theme-color" content="#ffffff"> | |||
<title>Commit - Developer tooling for the Frappeverse</title> | |||
<script type="module" crossorigin src="/assets/commit/dashboard/assets/index-2eacee2d.js"></script> | |||
<link rel="stylesheet" href="/assets/commit/dashboard/assets/index-31456e03.css"> | |||
<script type="module" crossorigin src="/assets/commit/dashboard/assets/index-63f9cb36.js"></script> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove this file not required
commit/www/dashboard.html
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove this file too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
keep this file dont remove this
Manage Branch Modal.. | ||
</DialogContent> | ||
</Dialog>} | ||
{isCreateAccess && |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
both button needs isCreateAccess right ?, then use it once
<DialogHeader> | ||
<DialogTitle>Manage Branches</DialogTitle> | ||
<DialogDescription> | ||
Select Branches to delete. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change this text as this modal will have multiple use Case like delete fetch repo
@@ -10,6 +10,7 @@ | |||
from commit.commit.code_analysis.apis import find_all_occurrences_of_whitelist | |||
from commit.commit.code_analysis.doctypes import get_doctypes_in_module, get_doctype_json | |||
from frappe.utils import now | |||
from frappe.app import handle_exception |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove this import
|
||
const handleProjectDelete = () => { | ||
|
||
deleteDoc("Commit Project", `${project.name}`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pass normally project.name
No description provided.