Skip to content

Next

  • Projects
  • Groups
  • Snippets
  • Help
  • Sign in / Register
Minds Frontend
Minds Frontend
  • Project overview
  • Repository
  • Issues 803
  • Merge Requests 52
  • CI / CD
  • Security & Compliance
  • Packages
  • Wiki
  • Snippets
  • Members
  • Collapse sidebar
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Minds
  • Minds FrontendMinds Frontend
  • Merge Requests
  • !622

Closed
Opened 6 days ago by Olivia Madrid@omadrid
Report abuse

WIP: Summary tab and chart refactor

Move chart into common so it can be reused for analytics charts, analytics summary tab minicharts and wallet redesign. Implement summary tab.

Will close #2170 #2053

Request to merge chart-refactor-2170 into master
Pipeline #93001789 failed for 4bea3399 on chart-refactor-2170
              Requires 3 more approvals from Devs, Deployers, and QA.
              Emiliano Balbuena
              Emiliano Balbuena
              Mark Harding
              Mark Harding
              Brian Hatchet
              Brian Hatchet
              Xander Miller
              Xander Miller
              All merge request dependencies have been merged (1 merged)

              Closed by Olivia Madrid 2 minutes ago

              The changes were not merged into master

              Did not close #2053 and #2170

              • Discussion 1
              • Commits 15
              • Pipelines 3
              • Changes 51
              • Olivia Madrid @omadrid added Platform::Browser Priority::1 - High Product::Analytics Sprint::10/23 - Quiet Quail Squad::Blue Type::Feature scoped labels 6 days ago

                added scoped labels

              • Olivia Madrid @omadrid added 1 commit 5 days ago

                added 1 commit

                • 95a24981 - (refactor): analytics chart into common

                Compare with previous version

              • Olivia Madrid @omadrid added 1 commit 4 days ago

                added 1 commit

                • 4bea3399 - (feat): add'l styles on analytics summary tab

                Compare with previous version

              • Mark Harding
                Mark Harding @markeharding · 11 hours ago
                Owner

                @omadrid looks like there are some failing tests here. Also is this now out of WIP?

              • Mark Harding @markeharding added MR::Requires Changes scoped label 11 hours ago

                added scoped label

              • Olivia Madrid @omadrid closed 2 minutes ago

                closed

              Please register or sign in to reply
              0 Assignees
              None
              None
              Milestone
              None
              Time tracking
              No estimate or time spent
              7
              Labels
              MR::Requires Changes Platform::Browser Priority::1 - High Product::Analytics Sprint::10/23 - Quiet Quail Squad::Blue Type::Feature
              Lock merge request
              Unlocked
              10
              10 participants
              user avatar
              Juan Manuel Solaro
              user avatar
              Martin Santangelo
              user avatar
              Emiliano Balbuena
              user avatar
              Marcelo Rivera
              user avatar
              Mark Harding
              user avatar
              Ben Hayward
              user avatar
              Guy Thouret
              Reference: minds/front!622