Jose Ivan Vargas (325140c2) at 17 Mar 17:21
Add CTA Set up CI pipeline button
... and 1418 more commits
Jose Ivan Vargas (d1093bb0) at 17 Mar 17:19
Merge branch 'harvie/artifact-download' into 'master'
... and 1 more commit
Tiny UI improvements to make my life easier one click at a time:
Before:
After:
Before:
After:
@csaez-ext I have replies and updates to your items
When I click
Set up CI pipelineduo opens with this prompt on the chatAnalyze this repository and generate a CI/CD pipelineCould we change the prompt to beAnalyze repository to create a working and fully functional pipeline
Done
After the prompt is sent, Duo attempts to respond, but then nothing happens. Is this connected with the AI Agent we are building, or isn't it connected yet?
Chances are this is your GDK running into issues when using AI features. The Duo Chat should use the default agent when the Pipeline Authoring agent is not available, I'm pretty sure there's something that's happening on our GDK that's now allowing you to see the responses.
For comparison this is my GDK
- Currently, when there isn't enough horizontal space to fit all actions in the repository toolbar (branch selector, Find file, Set up CI Pipeline, Code), the "Set up CI Pipeline" button is dropped entirely, while the other buttons remain in a weird order. The correct behavior should be - when the toolbar runs out of space, lower-priority actions should collapse into the existing overflow menu (the
...button). "Set up CI Pipeline" should be one of the hidden ones, but still accessible, not removed.
Now this one is tricky this weird order that you mention because there are multiple items in this page owned by multiple teams, changing this to set the button as a higher priority than others we risk causing issues to other teams. I'd suggest we stick to this order as I followed what other teams have done which is any item that gets added here will not show up on small to mobile viewports as the default options e.g. Find File, Code, Add File are the essential options that have to remain available in that order; For example the lock file and lock directory buttons will remain hidden even if we consider them higher priority as they're not part of the essentials
What do you think?
- Project information should appear below the repository toolbar On desktop, Project information lives in the right sidebar, it's supplementary content. When the layout collapses to a single column, the repository toolbar (branch selector, Find file, Set up CI Pipeline, Code) should come first since it's what users act on. Project information should follow below it.
I think this warrants a larger discussion with the larger UX team and the teams that own parts of this page, the rationale is sound but I'd suggest we skip these changes for now as we risk changing the expectations of other teams without input. Not to mention that this will make this small MR larger
Let me know if you have any further comments or questions!
Let me know what's the final decision on the button title, Setup up GitLab CI/CD is consistent with the branding and communication style we have presented for the feature for the longest time
frontend looks good to me, @syarynovskyi let me know if you need help merging this one
Update Premium billing page to adapt trial flow changes
| Before | After |
|---|---|
![]() |
![]() |
premium./-/billings page.Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Related to #587958
Jose Ivan Vargas (28809a98) at 17 Mar 16:25
Jose Ivan Vargas (5e8124b5) at 17 Mar 16:24
Merge branch '589262-add-missing-schedule-data' into 'master'
... and 1 more commit
Fix pipeline secondary link
Fixes issue where schedule link was not showing up due to missing data in GraphQL.
Changelog: fixed
| Before | After |
|---|---|
![]() |
![]() |
Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Related to #589262
This looks good to me, thank you for your contribution @harvie. Setting this to auto-merge
Jose Ivan Vargas (5e169ac3) at 17 Mar 15:56
Jose Ivan Vargas (f149653b) at 17 Mar 15:56
Merge branch 'renovate/dompurify-3.x-lockfile' into 'main'
... and 1 more commit
This MR contains the following updates:
| Package | Type | Update | Change |
|---|---|---|---|
| dompurify | devDependencies | patch | 3.3.2 -> 3.3.3 |
MR created with the help of gitlab-org/frontend/renovate-gitlab-bot
v3.3.3: DOMPurify 3.3.3
This MR has been generated by Renovate Bot.
Tiny UI improvements to make my life easier one click at a time:
Before:
After:
Before:
After:
Fix pipeline secondary link
Fixes issue where schedule link was not showing up due to missing data in GraphQL.
Changelog: fixed
| Before | After |
|---|---|
![]() |
![]() |
Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Related to #589262
The feature flag is owned by groupgitaly. @jcaigitlab For your review
@fforster Do you still need Drew and I assigned to this issue or can we defer to John for this one?
frontend looks good to me, let me know if you need help merging this one @pburdette