SpotiByeAds icon indicating copy to clipboard operation
SpotiByeAds copied to clipboard

Corrected - Angular 11 Product Dashboard Changes

Open VivekBuzruk opened this issue 3 years ago • 6 comments

1] Angular 8 -> Angular 11. Apart from package.json, few config. files like angular.json are also modified. Many files were modified to handle Angular 11 changes.

2] Little improvement to Build, Static Code and Commit Widget appearance -- Change of Project selection dropDown to Static Code widget is hidden through a flag. Need to introduce configured use of UI. This DropDown selection change occupies too much screen space and requires better Look & Feel. Also there was a reference to one API which was changed as was not found in current source code. -- Also Added pie chart to Static Code widget

3] Initial code for Product Dashboard is uploaded. Need lot of improvement.

4] Font source downloaded and made available as part of local assets directory. This is required for internal usage within an organization.

5] Few changes are driven by TSLint warnings (like introducing double quote etc.).

Node version used / required : v12.13.0+

VivekBuzruk avatar Jun 18 '21 06:06 VivekBuzruk

This pull request introduces 30 alerts when merging 364c1fe5157792f980655814aaa7f79ff2716a25 into 00bf0083bb56593512cc73e41b8696e0aa1efb8a - view on LGTM.com

new alerts:

  • 28 for Unused variable, import, function or class
  • 2 for Useless assignment to local variable

lgtm-com[bot] avatar Jun 18 '21 06:06 lgtm-com[bot]

This pull request introduces 30 alerts when merging 89bb968d1c1f54d19789bed2fce935f48a555972 into 00bf0083bb56593512cc73e41b8696e0aa1efb8a - view on LGTM.com

new alerts:

  • 28 for Unused variable, import, function or class
  • 2 for Useless assignment to local variable

lgtm-com[bot] avatar Jun 23 '21 05:06 lgtm-com[bot]

Engine-Wagon (rather Engine-Wagon-Guard) layout is a layout planned for Product Dashboard, where Team's / Component's CI/CD layout will be represented (Engine being Component Info along with Quality dimensions followed various Environment stages, last being Production). This is WIP and at this stage displays Product Dashboard's few parameters.

I focused more on correctness through manual tests and unit tests need to be developed

Anyway, to correct "test-build" errors, I committed modifications to my local repository (https://github.com/VivekBuzruk/Hygieia/commit/89bb968d1c1f54d19789bed2fce935f48a555972). Can not generate PR as it may include earlier commits too.

Need to spend more time to assess validity of all unit tests.

VivekBuzruk avatar Jun 23 '21 05:06 VivekBuzruk

@Sandhya-Rajasabeson - Any other challenge? Planning to add few unit / commit tests to product dashboard, but will like to know your plan to integrate PR.

Thanks

VivekBuzruk avatar Jun 29 '21 11:06 VivekBuzruk

Hi @VivekBuzruk , Sandhya is no longer with Capital One, so I will be taking over this work. Over the next couple days I will review this PR and see how it fits in with the other open PRs, and will get back to you later this week/early next week. Feel free to reach out with any questions/comments!

Best, Ryan

tatlax3636 avatar Jul 14 '21 15:07 tatlax3636

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

:white_check_mark: VivekBuzruk
:x: mend-bolt-for-github[bot]
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Jul 11 '22 19:07 CLAassistant