Drop namespaces_on_ldap_sync_last_successful_update_at for Gitlab.com

See merge request !135040



Merged-by: Gregory Havenga <11164960-ghavenga@users.noreply.gitlab.com>
Approved-by: Terri Chu <tchu@gitlab.com>
Approved-by: Gregory Havenga <11164960-ghavenga@users.noreply.gitlab.com>
Co-authored-by: Prabakaran Murugesan <pmurugesan@gitlab.com>

Fix PhoneVerification::TelesignCallbacksController#notify returning 500s

Closes gitlab-org/modelops/anti-abuse/team-tasks#520

See merge request !136416



Merged-by: Pedro Pombeiro <noreply@pedro.pombei.ro>
Approved-by: Pedro Pombeiro <noreply@pedro.pombei.ro>
Approved-by: Hinam Mehra <hmehra@gitlab.com>
Co-authored-by: Eugie Limpin <elimpin@gitlab.com>

Eugie Limpin authored 1 year ago and Pedro Pombeiro committed 1 year ago

find_by doesn't return an ActiveRecord::Relation so when it eventually
returns nil (i.e. when it can't find a record) the enclosing scope
returns Users::PhoneNumberValidation.all (an ActiveRecord::Relation)
instead of nil as expected. This results to the early return to be
skipped and the call to record.user throws
"undefined method `user' for #<ActiveRecord::Relation" error.

Changelog: fixed
EE: true

Display project full path and remove namespace column from token table

See merge request !136423



Merged-by: Justin Ho Tuan Duong <hduong@gitlab.com>
Approved-by: Briley Sandlin <bsandlin@gitlab.com>
Approved-by: Justin Ho Tuan Duong <hduong@gitlab.com>
Reviewed-by: Justin Ho Tuan Duong <hduong@gitlab.com>
Co-authored-by: Mireya Andres <mandres@gitlab.com>

Remove usages of gl-sm-flex-direction-column util

See merge request !136248



Merged-by: Miguel Rincon <mrincon@gitlab.com>
Approved-by: Elwyn Benson <ebenson@gitlab.com>
Approved-by: Lukas 'Eipi' Eipert <leipert@gitlab.com>
Approved-by: Miguel Rincon <mrincon@gitlab.com>
Reviewed-by: Elwyn Benson <ebenson@gitlab.com>
Co-authored-by: Paul Gascou-Vaillancourt <paul.gascvail@gmail.com>

Remove superseded line as the other notice is now moved back

See merge request gitlab-org/gitlab!136156



Merged-by: Lorena Ciutacu <lciutacu@gitlab.com>
Approved-by: Rutger Wessels <rwessels@gitlab.com>
Approved-by: Lorena Ciutacu <lciutacu@gitlab.com>
Approved-by: Christina Lohr <clohr@gitlab.com>
Co-authored-by: Thong Kuah <tkuah@gitlab.com>

Fixes NonOwnerAlertComponent auth check

See merge request gitlab-org/gitlab!136557



Merged-by: Max Woolf <mwoolf@gitlab.com>
Approved-by: Mohamed Hamda <mhamda@gitlab.com>
Approved-by: Max Woolf <mwoolf@gitlab.com>
Reviewed-by: Vijay Hawoldar <vhawoldar@gitlab.com>
Reviewed-by: Mohamed Hamda <mhamda@gitlab.com>
Co-authored-by: Vijay Hawoldar <vhawoldar@gitlab.com>

Use selected text in base prompt if present

See merge request !136459



Merged-by: Gosia Ksionek <mksionek@gitlab.com>
Approved-by: Gosia Ksionek <mksionek@gitlab.com>
Reviewed-by: Gosia Ksionek <mksionek@gitlab.com>
Reviewed-by: Nicolas Dular <ndular@gitlab.com>
Co-authored-by: Jan Provaznik <jprovaznik@gitlab.com>

Jan Provaznik authored 1 year ago and Gosia Ksionek committed 1 year ago

If selected text is present, prefer it instead of referer's blob.

Update the GitLab for Slack app docs

See merge request !136687



Merged-by: Lorena Ciutacu <lciutacu@gitlab.com>
Approved-by: Lorena Ciutacu <lciutacu@gitlab.com>
Reviewed-by: Lorena Ciutacu <lciutacu@gitlab.com>
Co-authored-by: Ashraf Khamis <akhamis@gitlab.com>

Re-orchestrate updating NuGet package flow from downloaded package file

See merge request gitlab-org/gitlab!134279



Merged-by: David Fernandez <dfernandez@gitlab.com>
Approved-by: Radamanthus Batnag <rbatnag@gitlab.com>
Approved-by: David Fernandez <dfernandez@gitlab.com>
Reviewed-by: David Fernandez <dfernandez@gitlab.com>
Reviewed-by: Moaz Khalifa <mkhalifa@gitlab.com>
Co-authored-by: moaz-khalifa <mkhalifa@gitlab.com>

Merge branch '429410-new-organization-form-swap-mock-graphql-mutation-for-real-mutation' into 'master' 

Use real GraphQL mutation when creating a new organization

See merge request !136147



Merged-by: Michael Lunøe <michael.lunoe@gmail.com>
Approved-by: Camellia X Yang <xyang@gitlab.com>
Approved-by: Ammar Alakkad <aalakkad@gitlab.com>
Approved-by: Michael Lunøe <michael.lunoe@gmail.com>
Reviewed-by: Ammar Alakkad <aalakkad@gitlab.com>
Reviewed-by: Michael Lunøe <michael.lunoe@gmail.com>
Co-authored-by: Peter Hegman <phegman@gitlab.com>

Peter Hegman authored 1 year ago and Michael Lunøe committed 1 year ago

Previously using mock data

Serve pages deployments without going through PagesMetadatum

See merge request !135993



Merged-by: Adam Hegyi <ahegyi@gitlab.com>
Approved-by: Désirée Chevalier <dchevalier@gitlab.com>
Approved-by: Janis Altherr <jaltherr@gitlab.com>
Approved-by: Sylvester Chin <schin@gitlab.com>
Approved-by: Vitali Tatarintev <vtatarintev@gitlab.com>
Approved-by: Adam Hegyi <ahegyi@gitlab.com>
Reviewed-by: Kassio Borges <kborges@gitlab.com>
Reviewed-by: Adam Hegyi <ahegyi@gitlab.com>
Reviewed-by: Sylvester Chin <schin@gitlab.com>
Co-authored-by: Kassio Borges <kassioborgesm@gmail.com>

Kassio Borges authored 1 year ago and Adam Hegyi committed 1 year ago

With the work to introduce [Gitlab Pages Multiple Versions], a project
now have to support multiple `PagesDeployment` at the same time.

For this reason, instead of using
`ProjectPagesMetadatum#pages_deployment_id` to identify valid deployment
the `PagesDeployment#deleted_at` field was [introduced].

For now, we're still serving a single pages version, to avoid changing
the current behavior, but instead of looking it up through
`ProjectPagesMetadatum#pages_deployment_id`, we're using the
`project.pages_deployments.main_version` to identify a project active
pages deployment.

This is an extraction of the [MR that introduce the multiple version
serving]. It was decided to not serve multiple versions on this commit
to avoid extra complexity on this, already, large change.

[Gitlab Pages Multiple Versions]: &10914
[introduce]: !130848
[MR that introduce the multiple version serving]: !133261

Related to: #416494

Changelog: changed

Add note explaining latest label for component releases

See merge request !136661



Merged-by: Lorena Ciutacu <lciutacu@gitlab.com>
Approved-by: Lorena Ciutacu <lciutacu@gitlab.com>
Reviewed-by: Fabio Pitino <fpitino@gitlab.com>
Reviewed-by: Lorena Ciutacu <lciutacu@gitlab.com>
Co-authored-by: Marcel Amirault <mamirault@gitlab.com>

Add version text for removing repository_storages API attribute

See merge request !136658



Merged-by: Jon Glassman <jglassman@gitlab.com>
Co-authored-by: Evan Read <eread@gitlab.com>

Reuse ClickHouseTestHelpers module in ClickHouseTestRunner

See merge request !136571



Merged-by: Max Woolf <mwoolf@gitlab.com>
Approved-by: Max Woolf <mwoolf@gitlab.com>
Approved-by: Adam Hegyi <ahegyi@gitlab.com>
Co-authored-by: Pedro Pombeiro <noreply@pedro.pombei.ro>

Mention ClickHouse rake tasks in documentation

See merge request !136579



Merged-by: Max Woolf <mwoolf@gitlab.com>
Reviewed-by: Pedro Pombeiro <noreply@pedro.pombei.ro>
Reviewed-by: Max Woolf <mwoolf@gitlab.com>
Co-authored-by: Pedro Pombeiro <noreply@pedro.pombei.ro>

Update searching in GitLab

See merge request !136680



Merged-by: Fiona Neill <fneill@gitlab.com>
Approved-by: Fiona Neill <fneill@gitlab.com>
Co-authored-by: Ashraf Khamis <akhamis@gitlab.com>

Update Workspaces documentation

See merge request !136673



Merged-by: Fiona Neill <fneill@gitlab.com>
Approved-by: Fiona Neill <fneill@gitlab.com>
Co-authored-by: Ashraf Khamis <akhamis@gitlab.com>

Show Chat button only when Chat can be used

See merge request !135548



Merged-by: Shinya Maeda <shinya@gitlab.com>
Approved-by: Shinya Maeda <shinya@gitlab.com>
Reviewed-by: Shinya Maeda <shinya@gitlab.com>
Reviewed-by: Gosia Ksionek <mksionek@gitlab.com>
Co-authored-by: Jessie Young <jessieyoung@gitlab.com>

Jessie Young authored 1 year ago and Shinya Maeda committed 1 year ago

* Previously, the logic would Chat button on all pages of gitlab.com
  to any user that can use Chat via any
  group.
* This MR updates the logic to be more fine-grained.
* A user will only see Chat if they are within the resource hierarchy of
  a Group that enables AI features that they are a member of.
* If the user is on a non-group/project resource, such as the Todos
  Dashboard, and they have AI features enabled via a Group membership,
  they will also see the Chat button (to ask Documentation questions)
* This is behind the `tanuki_bot_breadcrumbs_entry_point` feature flag
* This updates the logic for the Chat button only, not the Chat access
  available via the Help menu in the sidebar, which is written in Vue
  and can either stay as-is or be updated separately.
* See issue description for full details on what the expected behavior
  is and the rationale: #428541

Do not use rubocop server on lefthook calls to rubocop

See merge request !136632



Merged-by: Peter Leitzen <pleitzen@gitlab.com>
Approved-by: Peter Leitzen <pleitzen@gitlab.com>
Co-authored-by: Chad Woolley <cwoolley@gitlab.com>

Fix Google reCAPTCHA form submit button not visible in pipelines CC form

See merge request !136295



Merged-by: Jannik Lehmann <jlehmann@gitlab.com>
Approved-by: Angelo Gulina <agulina@gitlab.com>
Approved-by: Jannik Lehmann <jlehmann@gitlab.com>
Reviewed-by: Angelo Gulina <agulina@gitlab.com>
Co-authored-by: Eugie Limpin <elimpin@gitlab.com>
Co-authored-by: Angelo Gulina <agulina@gitlab.com>

Eugie Limpin authored 1 year ago and Jannik Lehmann committed 1 year ago

To accommodate Google reCAPTCHA challenge form, we set the height of the
iframe to 480px. Without doing this, the iframe height is insufficient
to display the challenge form in its entirety and the submit button of
the form is not visible to the user and cannot be clicked.

Changelog: fixed
EE: true

E2E: Quarantine instance audit streaming spec

See merge request !136675



Merged-by: Andrejs Cunskis <acunskis@gitlab.com>
Approved-by: Andrejs Cunskis <acunskis@gitlab.com>
Co-authored-by: Sanad Liaquat (Personal) <sliaquat@gitlab.com>

doc(api): Add NGitLab to C# clients list

See merge request !136544



Merged-by: Martin Wortschack <mwortschack@gitlab.com>
Approved-by: Martin Wortschack <mwortschack@gitlab.com>
Approved-by: Evan Read <eread@gitlab.com>
Co-authored-by: PMExtra <pm@jubeat.net>

Dequarantine CS test but leave out of smoke suite

See merge request gitlab-org/gitlab!136644



Merged-by: Ramya Authappan <rauthappan@gitlab.com>
Approved-by: Ramya Authappan <rauthappan@gitlab.com>
Co-authored-by: Mark Lapierre <mlapierre@gitlab.com>

Support Microsoft SAML attribute names by default

See merge request !135770



Merged-by: Aboobacker MK <akarakath@gitlab.com>
Approved-by: Smriti Garg <sgarg@gitlab.com>
Co-authored-by: Drew Blessing <drew@gitlab.com>

Drew Blessing authored 1 year ago and Aboobacker MK committed 1 year ago

Add default attribute_statements to support Microsoft SAML
attribute format. Microsoft uses a URL format by default. Prior
to this change self-managed instances had to manually configure
attribute statements and SaaS users had to add custom attributes
to their SAML responses. This makes both self-managed and SaaS
work out of the box with Microsoft.

Changelog: changed

Fixed "Time to SLA" column sort UI behavior

See merge request !133382



Merged-by: Tristan Read <tread@gitlab.com>
Approved-by: Pedro Moreira da Silva <hi@pedroms.com>
Approved-by: Tristan Read <tread@gitlab.com>
Co-authored-by: Rene Padillo <developer@renesansz.me>