Skip to content

Updating Whats New DuoProDiscover links with new doc locations

Buck O'Leary requested to merge 462651-why-duo-doc-link-updates into master

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:

  1. 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?

  1. Start the GDK and navigate to http://localhost:3000/groups/#{your-test-group}/-/add_ons/discover_duo_pro

  2. Click links and verify that the locations have been correctly updated

Related to #462651

Edited by Buck O'Leary

Merge request reports