| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
This commit also unifies layout structure
and remove no_container flag
|
|
|
|
| |
We're reversing the deprecation due to user feedback
|
|\
| |
| |
| |
| | |
Expand textarea for CA cert in cluster form
See merge request gitlab-org/gitlab-ce!32508
|
| |
| |
| |
| |
| | |
The current default is too small
for a user to see what they have pasted in.
|
|\ \
| | |
| | |
| | |
| | | |
Backport EE changes for built-in project template test
See merge request gitlab-org/gitlab-ce!32358
|
| |/ |
|
|\ \
| | |
| | |
| | |
| | | |
Update views for testability and update page objects
See merge request gitlab-org/gitlab-ce!32451
|
| |/ |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Fix Peek on Puma
Closes #66528
See merge request gitlab-org/gitlab-ce!32213
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Previously, we called the `peek_enabled?` method like so:
prepend_before_action :set_peek_request_id, if: :peek_enabled?
Now we don't have a `set_peek_request_id` method, so we don't need that
line. However, the `peek_enabled?` part had a side-effect: it would also
populate the request store cache for whether the performance bar was
enabled for the current request or not.
This commit makes that side-effect explicit, and replaces all uses of
`peek_enabled?` with the more explicit
`Gitlab::PerformanceBar.enabled_for_request?`. There is one spec that
still sets `SafeRequestStore[:peek_enabled]` directly, because it is
contrasting behaviour with and without a request store enabled.
The upshot is:
1. We still set the value in one place. We make it more explicit that
that's what we're doing.
2. Reading that value uses a consistent method so it's easier to find in
future.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Peek's `Peek.request_id` method doesn't work well with a multi-threaded
server and concurrent requests, because requests can 'steal' another
request's ID, or unset it before it was due.
The upstream change resolves this; the commit here is just to ensure
that GitLab works with that upstream change, mostly by not using
`Peek.request_id` any more (as the method doesn't exist).
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
Add CSS & JS global flags to represent browser and platform on login pages
See merge request gitlab-org/gitlab-ce!32521
|
| | |/
| |/|
| | |
| | |
| | |
| | | |
- Adds CSS classes to represent browser & platform
- Adds JS flags to `gl` object to represent
browser & platform
|
|/ /
| |
| |
| |
| |
| | |
This commit adds a warning message that is always displayed that informs
users that their GitLab Pages sites will take a bit of time before they
can be accessed after being deployed for the first time.
|
| |
| |
| |
| |
| |
| |
| | |
- Use "results" instead of "blobs", "wiki blobs", "snippet blobs"
- Use "comments" instead of "notes"
- Use correct pluralization
- Don't add "1 - 10 of" if there's only one page
|
|\ \ |
|
| |\ \
| | | |
| | | |
| | | |
| | | | |
Require a captcha after unique failed logins from the same IP
See merge request gitlab/gitlabhq!3270
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Add method to store session ids by ip
Add new specs for storing session ids
Add cleaning up records after login
Add retrieving anonymous sessions
Add login recaptcha setting
Add new setting to sessions controller
Add conditions for showing captcha
Add sessions controller specs
Add admin settings specs for login protection
Add new settings to api
Add stub to devise spec
Add new translation key
Add cr remarks
Rename class call
Add cr remarks
Change if-clause for consistency
Add cr remarks
Add code review remarks
Refactor AnonymousSession class
Add changelog entry
Move AnonymousSession class to lib
Move store unauthenticated sessions to sessions controller
Move link to recaptcha info
Regenerate text file
Improve copy on the spam page
Change action filter for storing anonymous sessions
Fix rubocop offences
Add code review remarks
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Make it harder to delete issuables accidentally
Closes #62387
See merge request gitlab-org/gitlab-ce!32376
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Previously submitting a DELETE request to an issuable URL would be
enough to destroy it, but this should require human confirmation. We
now require that the `destroy_confirm` parameter is set to a truthy
value before this can complete.
In addition, we log a Sentry error if a deletion arrived without
confirmation.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/62387
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Bring text mail for new issue & MR more in line
See merge request gitlab-org/gitlab-ce!32254
|
| | |_|_|/
| |/| | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Make these templates for mail notifications for a new issue and a new
merge request more similar.
Also remove some leading whitespace in text mails.
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Externalize strings from 'app/views/projects/services'
See merge request gitlab-org/gitlab-ce!32402
|
| | |/ / /
| |/| | |
| | | | |
| | | | |
| | | | | |
Signed-off-by: Tao Wang <twang2218@gmail.com>
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
| | | | | |
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Fix missing .btn-success class on the following buttons:
* 'Create milestone' button
* Secondary button on profile tabs
`btn-create` was an old class which was was removed in !23232
and replaced with `btn-success`,
but these two seem to have been missed in the process.
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Remove unnecessary CSS class
Closes #66163
See merge request gitlab-org/gitlab-ce!32320
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | | |
Add E2E spec to test that issue closes with commit message
|
| | | | |
| | | | |
| | | | |
| | | | | |
This reverts commit af5242ecb682189c5d8276e1ab1ffe5ce844f2e5.
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
'56130-operations-environments-shows-incorrect-deployment-date-for-manual-deploy-jobs' into 'master'
Show correct deployment date for a manual job
Closes #56130
See merge request gitlab-org/gitlab-ce!32072
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Only display time if the job has deployed
Update JS tests for displaying job deployment time
Simplify boolean check in deployedDate
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
- Extract MR fields for notes into a separate serializer
- Check if pipelines are empty via count
|
| | | | | | |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Add changelog entry
Remove unnecessary test checking for form value
Translations updated for gitlab.pot
Use proper format on changelog entry
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
This also restructures how and where the configuration for
Snowplow lives.
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Add Issue and Merge Request titles to Todo items
Closes #63488
See merge request gitlab-org/gitlab-ce!30435
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Only displays the todo body if the todo has a note.
This is to avoid redundant Issue or Merge Request titles
displayed both in the Todo title and body.
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
CE Backport of EE changes
See merge request gitlab-org/gitlab-ce!32203
|
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Add qa selector elements and files from the EE branch
|
| | | | | | | | |
|
|\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Add QA specs for using IP rate limits
See merge request gitlab-org/gitlab-ce!32177
|
| | |_|_|_|/ / /
| |/| | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
This will help prevent issues such as
https://gitlab.com/gitlab-org/gitlab-ce/issues/66449.
|
|\ \ \ \ \ \ \ \
| |/ / / / / / /
|/| | | | | | |
| | | | | | | |
| | | | | | | | |
Update view for testability and update page object
See merge request gitlab-org/gitlab-ce!32224
|
| | |_|_|_|/ /
| |/| | | | | |
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Fix Piwik not working
Closes #66627
See merge request gitlab-org/gitlab-ce!32234
|
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Due to indentation, the changes in
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/31402 broke the
templating of Piwik script header.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/66627
|
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
This reverts merge request !31826
|
|\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
apps: remove the too big "Mirror a repository" header
Closes #66445
See merge request gitlab-org/gitlab-ce!32097
|
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/66445
|