Age | Commit message (Collapse) | Author |
|
totalNotes is only used to prerender a number of
skeleton containers until real notes are loaded
issuable.discussions makes multiple requests, so
too expensive for this
This commit uses mere notes for this and sends
actual totalNotes number if it's less than 10;
otherwise it sends 10 - it allows us to avoid
bunch of skeleton prerenderings, which are not
necessary since they doesn't fit into the whole
screen and disappear quite fast
|
|
Call QuickActionsService on Note update
Add support for notes which just contain
commands after editing
Return http status gone (410) if note was deleted
Temporary frontend addition so it is not
failing when a note is deleted
Move specs to shared examples
Fix rubocop style issue
Deleting note on frontend when status is 410
Use guard clause for note which got deleted
Simplified condition for nil note
This method should no longer be called
with nil note
Refactoring of execute method to reduce
complexity
Move errors update to delete_note method
Note is now deleted visually when it only
contains commands after update
Add expectation
Fix style issues
Changing action to fix tests
Add tests for removeNote and update
deleteNote expectations
|
|
|
|
Add keyboard shortcuts `p` and `n` to navigate duscussions.
|
|
Send a `persist_filter: false` param to backend when
opening links to notes and auto-switching to show
all notes
|
|
|
|
|
|
|
|
New API endpoint for merge request count
Updates all open tabs at the same time with one call
Restructured API response
API response changed to 401 if no current_user
Added API + JS specs
Fix for Static Check
Updated Count on Open/Close, Assign/Unassign of MR's
Checking if MR Count is refreshed
Added # frozen_string_literal: true to spec
Added Changelog
|
|
|
|
|
|
- all commits squashed to make danger review happy
|
|
|
|
|
|
|
|
|
|
|
|
- next-discuss button is always show unless there is only 1 discussion
- regroup buttons arrangements
|
|
Add test for checking output
|
|
|
|
|
|
|
|
Revert useless timeout added in karma spec
|
|
|
|
|
|
- Adds color and a tooltip to describe this new behavior
- Does not resolve if discussion is already resolved
- Adds an action `resolveDiscussion` to simplify `toggleResolveNote`
- Updates docs
https://gitlab.com/gitlab-org/gitlab-ce/issues/54405
|
|
- Moved discussion notes out of `NoteableDiscussion` component into a
new `DiscussionNotes` component
- Wrote Jest tests for the new `DiscussionNotes` component
- Updated Jest config for emojis fixtures
- Updated Karma tests `NoteableDiscussion` to match its new structure
- Convert `DiffDiscussions` tests to use Vue test utils
|
|
- added suggestions to mock data
- fixed props to be not required
|
|
Also removes specs related to the flag
|
|
Upgrade to latest Version
|
|
|
|
|
|
|
|
|
|
|
|
Transforms notes when the note was transformed in some other tab
|
|
Closes https://gitlab.com/gitlab-org/gitlab-ee/issues/9974, https://gitlab.com/gitlab-org/gitlab-ee/issues/9973, https://gitlab.com/gitlab-org/gitlab-ee/issues/9969
|
|
Refactor note_form_spec.js to use Vue test utils
See merge request gitlab-org/gitlab-ce!25571
|
|
|
|
Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
|
|
Adds discussion note style section under notes app from where
user can toggle discussion when they have selected a filter to show
only system notes.
|
|
|
|
Make Jest work with gitlab-ui
Closes #56689
See merge request gitlab-org/gitlab-ce!25440
|
|
|
|
Backports template and Vue changes from https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/9632 to CE
|
|
Move convertToDiscussion call from ReplyButton to NoteApp component
|
|
When converting a comment to a discussion
if the "Cancel" button is used to exit the new discussion note,
the comment would revert back to its initial state
|
|
Sets `updated_at` so that the transformed note is returned together with
the new reply
|
|
(cherry picked from commit 861f93e6fc72ef4adbc4ace3fd297382e07b619c)
|
|
Resolve "Tool Tip Hides Menu Item"
Closes #55209
See merge request gitlab-org/gitlab-ce!24285
|