Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
G
gitlab-ce
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
0
Merge Requests
0
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
Léo-Paul Géneau
gitlab-ce
Commits
a2b2f1ea
Commit
a2b2f1ea
authored
Jul 21, 2017
by
Fatih Acet
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
IssueNotesRefactor: Fix issue spinach tests.
parent
c09b5852
Changes
3
Show whitespace changes
Inline
Side-by-side
Showing
3 changed files
with
14 additions
and
5 deletions
+14
-5
app/assets/javascripts/notes/components/issue_comment_form.vue
...ssets/javascripts/notes/components/issue_comment_form.vue
+8
-3
app/assets/javascripts/notes/stores/issue_notes_store.js
app/assets/javascripts/notes/stores/issue_notes_store.js
+6
-1
features/project/issues/issues.feature
features/project/issues/issues.feature
+0
-1
No files found.
app/assets/javascripts/notes/components/issue_comment_form.vue
View file @
a2b2f1ea
...
@@ -90,7 +90,9 @@ export default {
...
@@ -90,7 +90,9 @@ export default {
this
.
discard
();
this
.
discard
();
}
}
})
})
.
catch
(
this
.
handleError
);
.
catch
(()
=>
{
this
.
discard
(
false
);
});
}
}
if
(
withIssueAction
)
{
if
(
withIssueAction
)
{
...
@@ -109,12 +111,15 @@ export default {
...
@@ -109,12 +111,15 @@ export default {
$
(
`.js-btn-issue-action.
${
btnClass
}
:visible`
).
trigger
(
'
click
'
);
$
(
`.js-btn-issue-action.
${
btnClass
}
:visible`
).
trigger
(
'
click
'
);
}
}
},
},
discard
()
{
discard
(
shouldClear
=
true
)
{
// `blur` is needed to clear slash commands autocomplete cache if event fired.
// `blur` is needed to clear slash commands autocomplete cache if event fired.
// `focus` is needed to remain cursor in the textarea.
// `focus` is needed to remain cursor in the textarea.
this
.
$refs
.
textarea
.
blur
();
this
.
$refs
.
textarea
.
blur
();
this
.
$refs
.
textarea
.
focus
();
this
.
$refs
.
textarea
.
focus
();
if
(
shouldClear
)
{
this
.
note
=
''
;
this
.
note
=
''
;
}
},
},
setNoteType
(
type
)
{
setNoteType
(
type
)
{
this
.
noteType
=
type
;
this
.
noteType
=
type
;
...
...
app/assets/javascripts/notes/stores/issue_notes_store.js
View file @
a2b2f1ea
...
@@ -251,7 +251,12 @@ const actions = {
...
@@ -251,7 +251,12 @@ const actions = {
context
.
commit
(
'
removePlaceholderNotes
'
);
context
.
commit
(
'
removePlaceholderNotes
'
);
return
res
;
return
res
;
});
})
.
catch
(()
=>
{
const
msg
=
'
Your comment could not be submitted! Please check your network connection and try again.
'
;
Flash
(
msg
,
'
alert
'
,
$
(
noteData
.
flashContainer
));
context
.
commit
(
'
removePlaceholderNotes
'
);
})
},
},
poll
(
context
)
{
poll
(
context
)
{
const
{
notesPath
}
=
$
(
'
.js-notes-wrapper
'
)[
0
].
dataset
;
const
{
notesPath
}
=
$
(
'
.js-notes-wrapper
'
)[
0
].
dataset
;
...
...
features/project/issues/issues.feature
View file @
a2b2f1ea
...
@@ -46,7 +46,6 @@ Feature: Project Issues
...
@@ -46,7 +46,6 @@ Feature: Project Issues
Given
I visit issue page
"Release 0.4"
Given
I visit issue page
"Release 0.4"
And
I leave a comment like
"XML attached"
And
I leave a comment like
"XML attached"
Then
I should see comment
"XML attached"
Then
I should see comment
"XML attached"
And
I should see an error alert section within the comment form
@javascript
@javascript
Scenario
:
Visiting Issues after being sorted the list
Scenario
:
Visiting Issues after being sorted the list
...
...
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