| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
By pre-calculating the digests we can manually construct the emoji URLs,
removing the need for using Rails' asset URL helpers. The reason we
don't want to use these helpers for Emojis is two-fold:
1. Rails' image_url() method is slow, really slow. For one it _might_
have to calculate digests but it also performs a lot of other
intensive operations (judging by the source code and based on
measuring timings).
2. We have a lot of Emoji which coupled with the above can result in it
taking minutes to load Emoji autocomplete data.
Using this pre-calculation setup generating the digests takes around 7
seconds (including the time it takes to start Rails/Rake), and only
around 600 milliseconds to load _all_ the autocomplete data of a project
(measured locally).
This commit _does_ change the Emoji URLs from absolute to relative URLs
as these are much easier to generate.
To update the Emoji data simply run:
rake gemojione:digests
Then commit any changes.
Fixes gitlab-org/gitlab-ce#14009
|
|
|
|
| |
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
|\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Functional closed sidebar
Adds functionality to include the closed sidebar.
When sidebar is closed the details will be visible once changed in open sidebar.
This MR fixes some bugs and adds some features from https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/3175
Fixes: #13594
See merge request !3411
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
|/ |
|
|\
| |
| |
| |
| |
| |
| | |
Fixed issue with notification settings not saving
Closes #14736
See merge request !3452
|
|/
|
|
| |
Closes #14736
|
|\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Remove redundant `require`s from Banzai files
We're trying to avoid circular dependency errors.
Related: https://gitlab.com/gitlab-org/gitlab-ce/issues/13521
Sentry: https://sentry.gitlap.com/gitlab/gitlabcom/issues/8/
See merge request !3391
|
| |
| |
| |
| | |
We're trying to avoid circular dependency errors.
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Dropdown bug fixes
- Added back upcoming to milestone filter
![Screen_Shot_2016-03-23_at_16.45.22](/uploads/8c31eff7e0984aad17881f6397ff17cd/Screen_Shot_2016-03-23_at_16.45.22.png)
- Fixed bug that would cause the currently selected value to disappear on filters
- Fixed bug that throw an error when filtering by upcoming when there is only a milestone in the past
Closes #14536
See merge request !3368
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | | |
The frontend will then always use the name as the ID - like previous
|
| | | |
|
| | |
| | |
| | |
| | | |
Fixes tests
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | | |
Added back upcoming to milestone filter
Fixed bug that would cause the currently selected value to disappear on filters
Fixed bug that throw an error when filtering by upcoming when there is only a milestone in the past
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Gracefully handle notes on deleted commits in merge requests
Closes #3250
See merge request !3402
|
| | | |
| | | |
| | | |
| | | | |
Closes #3250
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
'mariusz_jachimowicz/gitlab-ce-i_14415_expose_label_description' into 'master'
api - expose label description
Resolves #14415
See merge request !3314
|
| | |_|/
| |/| | |
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Ensuring Merge Request API returns boolean values for work_in_progress
Fixes #14692.
See merge request !3432
|
| | | | |
| | | | |
| | | | |
| | | | | |
Fixes #14692
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Handle nil descriptions in Slack issue messages
Closes #14676
See merge request !3420
|
| | |_|/ /
| |/| | |
| | | | |
| | | | | |
Closes #14676
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Updated gitlab-shell to 2.7
This is done to include the changes introduced in
https://gitlab.com/gitlab-org/gitlab-shell/merge_requests/42.
See merge request !3437
|
|/ / / / /
| | | | |
| | | | |
| | | | |
| | | | | |
This is done to include the changes introduced in
https://gitlab.com/gitlab-org/gitlab-shell/merge_requests/42.
|
|\ \ \ \ \
| |_|_|_|/
|/| | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Use raw SQL instead of Rails models in migrations where appropriate
Closes #14694.
See merge request !3429
|
| | | | |
| | | | |
| | | | |
| | | | | |
Closes #14694.
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Fix order of steps to prevent PostgreSQL errors when running migration
Supersedes https://github.com/gitlabhq/gitlabhq/pull/10100.
/cc @stanhu @rodrigomanhaes
See merge request !3355
|
| | | | | | |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Fix error when visiting CI root path
Closes #14528, closes #14687
See merge request !3377
|
| | | | | | | |
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Closes #14528
|
|\ \ \ \ \ \ \ |
|
| |\ \ \ \ \ \ \
| | |_|_|/ / / /
| |/| | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Ensure private project snippets are not viewable by unauthorized people
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/14607
See merge request !1946
|
| | | | | | | | |
|
| | |/ / / / /
| | | | | | |
| | | | | | |
| | | | | | | |
Fix https://gitlab.com/gitlab-org/gitlab-ce/issues/14607.
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Remove tabstop from the WIP toggle links
[ci skip]
See merge request !3426
|
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
[ci skip]
|
|\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Fix dropdown line height
**Before**
![Screen_Shot_2016-03-28_at_5.54.37_PM](/uploads/67681da80b9104924083f13af3833553/Screen_Shot_2016-03-28_at_5.54.37_PM.png)
**After**
![Screen_Shot_2016-03-28_at_5.57.37_PM](/uploads/5df14c2d131cc88dbc75b87959f56c3c/Screen_Shot_2016-03-28_at_5.57.37_PM.png)
See merge request !3431
|