| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Consider the scenario:
1. The default visibility level is set to internal
2. A user attempts to create a private project within a private group
Previously this would always fail because default_value_for would
overwrite the private visibility setting, no matter what
visibility_level were specified. This was happening because
default_value_for was confused by the default value of 0 specified by
the database schema.
default_value_for attempts to assign the default value in the block by
checking whether the attribute has changed. The problem is that since
the default value by the database was 0, and the user requested 0, this
appeared as though no changes were made. As a result, default_value_for
would always overwrite the user's preference.
To fix this, we remove the use of default_value_for and only set the
visibility level to the default application setting when no preference
has been given at creation time.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/63158
|
|
|
|
|
|
|
|
|
|
|
| |
In production, we've seen the rendering times of the merge request
widget increase as a result of loading commit data. BatchLoader attempts
to call replace_methods on the lazy object, but this has a significant
performance penalty for modules that have many methods. Disabling this
mode (https://github.com/exAspArk/batch-loader/pull/45) appears to cut
load times by about 50% for MergeRequestsController#show.
Relates to https://gitlab.com/gitlab-com/gl-infra/infrastructure/issues/6941
|
|\
| |
| |
| |
| |
| |
| | |
Omit blocked admins from repository check e-mails
Closes #63019
See merge request gitlab-org/gitlab-ce!29507
|
| |
| |
| |
| |
| |
| |
| | |
Blocked admins should not receive notifications about failed repository
checks.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/63019
|
|\ \
| | |
| | |
| | |
| | | |
Use serialization for project boards
See merge request gitlab-org/gitlab-ce!29263
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | | |
- Add serializers and Grape::Entity
- Replace to_json
- Add specs
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Unnecessary offences include false positives as well as flagged
errors which have follow up issues in order to be addressed.
Not all issues have been addressed before this being submitted
in the spirit of results and iteration.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Opts to cache a full list of cached dashboards
to better manage removing items from the cache.
This also allows dashboards to be stored in the
cache that don't necessarily correspond to a
single dashboard yml.
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
Raise an error if creating the MergeRequestDiff fails
See merge request gitlab-org/gitlab-ce!29434
|
| | | | |
|
|\ \ \ \
| |/ / /
|/| | |
| | | |
| | | | |
Click file row in repository Vue app to view file
See merge request gitlab-org/gitlab-ce!29596
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Backport changes from 'Partially revert d9cb907c: "Avoid loading objects from DB in ES results"'
See merge request gitlab-org/gitlab-ce!29597
|
| | | | | |
|
| |/ / /
| | | |
| | | |
| | | | |
This reverts commit d9cb907c3e987363065136bafb2156e86bc5de26.
|
|/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Commit messages are not processed for references to issues when
creating the default branch on push. This was expected
behavior (probably to avoid performance problems when first pushing a
repository with thousands of commits). However, this is not an issue
because we always limit the number of commits to process to 100
regardless of whether we are creating the default branch or not.
|
| |/
|/|
| |
| | |
This is a no-op for CE.
|
| | |
|
| |
| |
| |
| | |
long words
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
'63010-visual-regression-in-12-0-line-numbers-not-lining-up-in-file-view' into 'master'
Resolve "Visual regression in 12.0: line numbers not lining up in file view"
Closes #63010
See merge request gitlab-org/gitlab-ce!29403
|
| | | |
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Fix "Resolve conflicts" button not appearing for some users
Closes #47954
See merge request gitlab-org/gitlab-ce!29535
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Previously the frontend assumed that the user had to be able to merge to
that project in order to resolve conflicts. However, this is overly
restrictive, as the user only has to be able to push to the source
branch. In fact, appending the text /conflicts to the merge request
would bring up the conflict resolution page.
This confusion happens when a project contains a protected branch that
only allows maintainers to push. Users with Developer access no longer
have permission to merge, but they still can create branches in that
project.
To fix this issue, we now loosen the permission check for the "Resolve
conflicts" button and only check for access to push to the source
branch. This is consistent with what the backend does in
MergeRequests::Conflicts::ListService#can_be_resolved_by?.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/47954
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Style popovers according to the design specs
See merge request gitlab-org/gitlab-ce!28375
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Separate use-case specific user-popover styles from
general popover styles.
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
- Apply new styles for header, padding, border, and fonts
- Remove bootstrap 3 backward compat code
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Switch charts to Prometheus API endpoint CE-3
Closes #58516
See merge request gitlab-org/gitlab-ce!29280
|
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Make API request for each chart
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Fix diverged branch locals
See merge request gitlab-org/gitlab-ce!29508
|
| | | | | | | |
|
|\ \ \ \ \ \ \
| | |/ / / / /
| |/| | | | |
| | | | | | |
| | | | | | | |
Switch charts to Prometheus API endpoint CE-2
See merge request gitlab-org/gitlab-ce!29226
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
First part of FE for Prometheus API
Dashboard endpoint fetches all info except for chart results
Renders empty groups after loading
|
| | | | | | | |
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
**Why?**
The ide_status_list will be used and extended in EE.
|
| | | | | | | |
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
'master'
Resolve "Remove unnecessary decimal places on chart axes"
Closes #59651
See merge request gitlab-org/gitlab-ce!29468
|
| | | | | | | | |
|
|\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Add EE backport for the admin view
See merge request gitlab-org/gitlab-ce!29268
|
| | | | | | | | | |
|
|\ \ \ \ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Moves table pagination component
Closes #62788
See merge request gitlab-org/gitlab-ce!29543
|
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Move the table paginaiton component under the
new pagination folder in vue shared components
and updates all the paths
|
|\ \ \ \ \ \ \ \ \ \
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Expose default_git_depth via project API
Closes #62908
See merge request gitlab-org/gitlab-ce!29353
|
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Enable Get and Update of ci_default_git_depth for
Project API.
Renaming Project#default_git_depth to :ci_default_git_depth
to give more context through the API usage.
Add API documentation
|
|\ \ \ \ \ \ \ \ \ \ \
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
Rename `BatchCommitLoader` to `BatchLfsOidLoader`
See merge request gitlab-org/gitlab-ce!29487
|
| | |_|/ / / / / / / /
| |/| | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Since we're not actually loading commits in that loader, but we're
loading blobs with LFS oids in batch and returning only the oids.
|