BigW Consortium Gitlab
Skip to content
Projects
Groups
Snippets
Help
This project
Loading...
Sign in / Register
Toggle navigation
G
gitlab-ce
Project
Overview
Details
Activity
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
Registry
Registry
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Commits
Issue Boards
Open sidebar
Forest Godfrey
gitlab-ce
Commits
96ede7f7
Commit
96ede7f7
authored
Jun 16, 2017
by
Fatih Acet
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
IssueNotesRefactor: Implement logic for delete action.
No backend request for now. It’s just client side logic.
parent
66f4af5c
Hide whitespace changes
Inline
Side-by-side
Showing
6 changed files
with
62 additions
and
9 deletions
+62
-9
issue_discussion.vue
app/assets/javascripts/notes/components/issue_discussion.vue
+1
-1
issue_note.vue
app/assets/javascripts/notes/components/issue_note.vue
+21
-2
issue_note_actions.vue
...ssets/javascripts/notes/components/issue_note_actions.vue
+8
-1
issue_system_note.vue
...assets/javascripts/notes/components/issue_system_note.vue
+2
-2
issue_notes_service.js
app/assets/javascripts/notes/services/issue_notes_service.js
+3
-0
issue_notes_store.js
app/assets/javascripts/notes/stores/issue_notes_store.js
+27
-3
No files found.
app/assets/javascripts/notes/components/issue_discussion.vue
View file @
96ede7f7
...
...
@@ -48,7 +48,7 @@ export default {
this
.
$store
.
commit
(
'toggleDiscussion'
,
{
discussionId
:
this
.
note
.
id
,
});
}
}
,
},
};
</
script
>
...
...
app/assets/javascripts/notes/components/issue_note.vue
View file @
96ede7f7
...
...
@@ -14,6 +14,7 @@ export default {
data
()
{
return
{
isEditing
:
false
,
isDeleting
:
false
,
};
},
components
:
{
...
...
@@ -26,11 +27,28 @@ export default {
author
()
{
return
this
.
note
.
author
;
},
classNameBindings
()
{
return
{
'is-editing'
:
this
.
isEditing
,
'disabled-content'
:
this
.
isDeleting
,
};
},
},
methods
:
{
editHandler
()
{
this
.
isEditing
=
true
;
},
deleteHandler
()
{
this
.
isDeleting
=
true
;
this
.
$store
.
dispatch
(
'deleteNote'
,
this
.
note
)
.
then
(()
=>
{
this
.
isDeleting
=
false
;
})
.
catch
(()
=>
{
this
.
isDeleting
=
false
;
});
},
formUpdateHandler
()
{
// console.log('update requested', data);
},
...
...
@@ -44,7 +62,7 @@ export default {
<
template
>
<li
class=
"note timeline-entry"
:class=
"
{ 'is-editing': isEditing }
">
:class=
"
classNameBindings
"
>
<div
class=
"timeline-entry-inner"
>
<div
class=
"timeline-icon"
>
<user-avatar-link
...
...
@@ -66,7 +84,8 @@ export default {
:canEdit=
"note.can_edit"
:canDelete=
"note.can_edit"
:reportAbusePath=
"note.report_abuse_path"
:editHandler=
"editHandler"
/>
:editHandler=
"editHandler"
:deleteHandler=
"deleteHandler"
/>
</div>
<issue-note-body
:note=
"note"
...
...
app/assets/javascripts/notes/components/issue_note_actions.vue
View file @
96ede7f7
...
...
@@ -26,6 +26,10 @@ export default {
type
:
Function
,
required
:
true
,
},
deleteHandler
:
{
type
:
Function
,
required
:
true
,
},
},
data
()
{
return
{
...
...
@@ -92,7 +96,10 @@ export default {
</a>
</li>
<li>
<a
class=
"js-note-delete"
>
<a
@
click
.
prevent=
"deleteHandler"
class=
"js-note-delete"
href=
"#"
>
<span
class=
"text-danger"
>
Delete comment
</span>
...
...
app/assets/javascripts/notes/components/issue_system_note.vue
View file @
96ede7f7
...
...
@@ -12,12 +12,12 @@ export default {
data
()
{
return
{
svg
:
iconsMap
[
this
.
note
.
system_note_icon_name
],
}
}
;
},
components
:
{
IssueNoteHeader
,
},
}
}
;
</
script
>
<
template
>
...
...
app/assets/javascripts/notes/services/issue_notes_service.js
View file @
96ede7f7
...
...
@@ -7,4 +7,7 @@ export default {
fetchNotes
(
endpoint
)
{
return
Vue
.
http
.
get
(
endpoint
);
},
deleteNote
(
endpoint
)
{
return
Vue
.
http
.
get
(
endpoint
);
},
};
app/assets/javascripts/notes/stores/issue_notes_store.js
View file @
96ede7f7
...
...
@@ -3,9 +3,7 @@
import
service
from
'../services/issue_notes_service'
;
const
findNoteObjectById
=
(
notes
,
id
)
=>
{
return
notes
.
filter
(
n
=>
n
.
id
===
id
)[
0
];
};
const
findNoteObjectById
=
(
notes
,
id
)
=>
notes
.
filter
(
n
=>
n
.
id
===
id
)[
0
];
const
state
=
{
notes
:
[],
...
...
@@ -26,6 +24,20 @@ const mutations = {
discussion
.
expanded
=
!
discussion
.
expanded
;
},
deleteNote
(
storeState
,
note
)
{
const
noteObj
=
findNoteObjectById
(
storeState
.
notes
,
note
.
discussion_id
);
if
(
noteObj
.
individual_note
)
{
storeState
.
notes
.
splice
(
storeState
.
notes
.
indexOf
(
noteObj
),
1
);
}
else
{
const
comment
=
findNoteObjectById
(
noteObj
.
notes
,
note
.
id
);
noteObj
.
notes
.
splice
(
noteObj
.
notes
.
indexOf
(
comment
),
1
);
if
(
!
noteObj
.
notes
.
length
)
{
storeState
.
notes
.
splice
(
storeState
.
notes
.
indexOf
(
noteObj
),
1
);
}
}
},
};
const
actions
=
{
...
...
@@ -40,6 +52,18 @@ const actions = {
new
Flash
(
'Something went wrong while fetching issue comments. Please try again.'
);
// eslint-disable-line
});
},
deleteNote
(
context
,
note
)
{
// FIXME: Implement request, remove fake delete timer...
return
service
.
deleteNote
(
`
${
document
.
location
.
href
}
.json`
)
.
then
(
res
=>
res
.
json
)
.
then
(()
=>
{
context
.
commit
(
'deleteNote'
,
note
);
})
.
catch
(()
=>
{
new
Flash
(
'Something went wrong while deleting your note. Please try again.'
);
// eslint-disable-line
});
},
};
export
default
{
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment