Updating Whats New DuoProDiscover links with new doc locations
What does this MR do and why?
Updates the "What's new in GitLab Duo Chat" section documentation links per discussion in this thread in the preceding MR.
All links were updated per this comment - after reviewing more in depth I realized that the "Chat from any location" link also needed to be updated to "user/gitlab_duo_chat/index" as well.
Links have been tested locally and are now pointing to the correct locations - screencaps attached.
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
Before | After |
---|---|
before-screen-recording | quick-after-screenrecording |
How to set up and validate locally
Make Duo Pro Discover show page available by following the steps in @jmontal's initial MR. Re-posting rails console commands for clarity / ease here:
-
bin/rails c
GitlabSubscriptions::AddOn.find_or_create_by(name: 'code_suggestions', description: GitlabSubscriptions::AddOn.descriptions[:code_suggestions])
GitlabSubscription.last.update(trial: true, trial_starts_on: 45.days.ago, trial_ends_on: 15.days.ago)
GitlabSubscriptions::AddOnPurchase.create(
subscription_add_on_id: GitlabSubscriptions::AddOn.find_by(name: 'code_suggestions').id,
namespace: Group.last,
started_at: 60.days.ago,
expires_on: Date.today,
trial: true,
quantity: 1,
purchase_xid: '1'
)
Caveats / Troubleshooting - depending on your current local environment you may need to create additional resources for those rails console commands to execute successfully (I believe I needed to create a GitlabSubscription in the console as well.)
If you're impatient and those steps don't fully get you there, you should also be able to force the rendering of the Duo Pro Discover page by modifying the following file like so - although be careful not to commit it.
ee/app/models/gitlab_subscriptions/trials/duo_pro.rb
def self.show_duo_pro_discover?(namespace, user)
return true
return false unless namespace.present?
return false unless user.present?
-
Start the GDK and navigate to http://localhost:3000/groups/#{your-test-group}/-/add_ons/discover_duo_pro
-
Click links and verify that the locations have been correctly updated
Related to #462651
Merge request reports
Activity
changed milestone to %17.4
requested review from @syarynovskyi and @gdoud
assigned to @bucoleary
added pipelinetier-1 label
removed pipelinetier-1 label
- A deleted user
added backend label
1 Warning You've made some app changes, but didn't add any tests.
That's OK as long as you're refactoring existing code,
but please consider adding any of the maintenancepipelines, maintenancerefactor, maintenanceworkflow, documentation, QA labels.1 Message CHANGELOG missing: If this merge request needs a changelog entry, add the
Changelog
trailer to the commit message you want to add to the changelog.If this merge request doesn't need a CHANGELOG entry, feel free to ignore this message.
Reviewer roulette
Category Reviewer Maintainer backend @sam.figueroa
(UTC+0)
@maddievn
(UTC+2)
Please refer to documentation page for guidance on how you can benefit from the Reviewer Roulette, or use the GitLab Review Workload Dashboard to find other available reviewers.
If needed, you can retry the
danger-review
job that generated this comment.Generated by
Dangerremoved backend label
added maintenancerefactor label
added typemaintenance label and removed typefeature label
added typefeature label and removed typemaintenance label
removed maintenancerefactor label
mentioned in merge request !161506 (merged)
added frontend label
removed frontend label
added pipeline:mr-approved label
added pipelinetier-2 label
- Resolved by Madelein van Niekerk
@maddievn Could you pls take over for maintainer review?
requested review from @maddievn and removed review request for @syarynovskyi
added pipelinetier-3 pipeline:run-e2e-omnibus-once labels and removed pipelinetier-2 label
Before you set this MR to auto-merge
This merge request will progress on pipeline tiers until it reaches the last tier: pipelinetier-3. We will trigger a new pipeline for each transition to a higher tier.
Before you set this MR to auto-merge, please check the following:
- You are the last maintainer of this merge request
- The latest pipeline for this merge request is pipelinetier-3 (You can find which tier it is in the pipeline name)
- This pipeline is recent enough (created in the last 8 hours)
If all the criteria above apply, please set auto-merge for this merge request.
See pipeline tiers and merging a merge request for more details.
- A deleted user
added backend label
E2E Test Result Summary
allure-report-publisher
generated test report!e2e-test-on-gdk:
test report for eec05583expand test summary
+------------------------------------------------------------------+ | suites summary | +-------------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +-------------+--------+--------+---------+-------+-------+--------+ | Create | 128 | 0 | 15 | 0 | 143 | ✅ | | Verify | 44 | 0 | 2 | 0 | 46 | ✅ | | Plan | 73 | 0 | 0 | 0 | 73 | ✅ | | Data Stores | 31 | 0 | 1 | 0 | 32 | ✅ | | Govern | 71 | 0 | 0 | 0 | 71 | ✅ | | Fulfillment | 2 | 0 | 0 | 0 | 2 | ✅ | | Package | 20 | 0 | 12 | 0 | 32 | ✅ | | Monitor | 8 | 0 | 0 | 0 | 8 | ✅ | | Manage | 1 | 0 | 1 | 0 | 2 | ✅ | | Secure | 3 | 0 | 0 | 0 | 3 | ✅ | | Release | 5 | 0 | 0 | 0 | 5 | ✅ | | Analytics | 2 | 0 | 0 | 0 | 2 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+ | Total | 388 | 0 | 31 | 0 | 419 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+
started a merge train
mentioned in commit f8e9377c
added workflowstaging-canary label and removed workflowverification label
added workflowcanary label and removed workflowstaging-canary label
added workflowstaging label and removed workflowcanary label
added workflowproduction label and removed workflowstaging label
added workflowpost-deploy-db-staging label and removed workflowproduction label
added workflowpost-deploy-db-production label and removed workflowpost-deploy-db-staging label
added releasedcandidate label
added releasedpublished label and removed releasedcandidate label