Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(process-explorer/frontend): Processes View with Angular Material #325

Merged

Conversation

agyen
Copy link
Contributor

@agyen agyen commented Aug 16, 2023

This PR introduces the first part of reimplementation of the Process Explorer Frontend with open-source technologies.
It's leveraging Angular Material instead of IgniteUI.

Features

  • Processes View (with mock data)
  • Side Navigation

@agyen agyen requested a review from a team August 16, 2023 12:03
@codecov
Copy link

codecov bot commented Sep 6, 2023

Codecov Report

Merging #325 (656ea83) into main (495ebe3) will increase coverage by 0.12%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #325      +/-   ##
==========================================
+ Coverage   74.62%   74.75%   +0.12%     
==========================================
  Files         155      155              
  Lines        5474     5474              
  Branches      233      233              
==========================================
+ Hits         4085     4092       +7     
+ Misses       1388     1381       -7     
  Partials        1        1              

see 3 files with indirect coverage changes

📢 Have feedback on the report? Share it here.

@kruplm kruplm changed the title Processes view with angular material feat(process-explorer/frontend): Processes View with Angular Material Sep 7, 2023
kruplm
kruplm previously approved these changes Sep 8, 2023
@agyen agyen dismissed kruplm’s stale review September 8, 2023 08:18

The merge-base changed after approval.

kruplm
kruplm previously approved these changes Sep 8, 2023
@agyen agyen dismissed kruplm’s stale review September 8, 2023 08:27

The merge-base changed after approval.

@kruplm kruplm closed this Sep 8, 2023
@kruplm kruplm reopened this Sep 8, 2023
@kruplm kruplm merged commit 605c56e into morganstanley:main Sep 8, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants