Skip to content
Snippets Groups Projects

Migrates buttons to Pajamas

Merged Nick Leonard requested to merge nl-migrate-dashboard-buttons into master
All threads resolved!

What does this MR do and why?

Migrates buttons on various dashboard pages to Pajamas components

Closes #384401 (closed), #384415 (closed), #384413 (closed)

Screenshots or screen recordings

Issue Location Before After
Snippets #384401 (closed) http://gdk.test:3000/dashboard/snippets image image
Projects #384415 (closed) http://gdk.test:3000/dashboard/projects image image
Activity #384413 (closed) http://gdk.test:3000/dashboard/activity image image

How to set up and validate locally

Snippets

  1. Navigate to http://gdk.test:3000/dashboard/snippets, migrated button is New snippet button at the top (must have >0 snippets to see this)

Projects

  1. Navigate to http://gdk.test:3000/dashboard/projects, migrated button is New project button at the top

Activity

  1. Navigate to http://gdk.test:3000/dashboard/activity, migrated button is the Subscribe RSS button near the header

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Nick Leonard added 1 commit

    added 1 commit

    • bcafa9db - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • mentioned in issue #384964 (closed)

  • Matthew Nearents approved this merge request

    approved this merge request

  • Matthew Nearents removed review request for @mnearents

    removed review request for @mnearents

  • Coung Ngo resolved all threads

    resolved all threads

  • Coung Ngo approved this merge request

    approved this merge request

  • Coung Ngo enabled an automatic merge when the pipeline for f72670aa succeeds

    enabled an automatic merge when the pipeline for f72670aa succeeds

  • merged

  • Coung Ngo mentioned in commit 7337b4f9

    mentioned in commit 7337b4f9

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading