Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
bastilian created a review on a pull request on RedHatInsights/compliance-frontend
LGTM! Thank you @LightOfHeaven1994!
bastilian created a comment on a pull request on RedHatInsights/vulnerability-ui
@johnsonm325 We should maybe update the fec packages in a separate PR. There are a lot of changes that aren't really related to changing to `singleSelect`.
bastilian created a review on a pull request on RedHatInsights/compliance-frontend
LGTM! Thank you @LightOfHeaven1994!
bastilian created a review on a pull request on RedHatInsights/frontend-components
LGTM! Thank you @regexowl for updating the ranges!
bastilian created a review on a pull request on RedHatInsights/insights-inventory-frontend
LGTM! Thank you for setting this up @gkarat!
bastilian created a comment on a pull request on RedHatInsights/insights-advisor-frontend
> that should be solved by https://github.com/RedHatInsights/insights-advisor-frontend/pull/1365/ ... this is #1365 🙃
bastilian created a review on a pull request on RedHatInsights/insights-inventory-frontend
LGTM! Thank you for fixing this @leSamo!
bastilian created a review on a pull request on RedHatInsights/patchman-ui
LGTM! OS filter appears to still work as expected! Thank you @jiridostal & @gkarat!
bastilian pushed 512 commits to master bastilian/patchman-ui
- SPM-1321: make advisory cards responsive 1bd11f9
- SPM-1287: make system status card respect only tags, os, systemPlatform 5fa217c
- sonarqube config 20e926b
- chore(deps): bump @patternfly/react-icons from 4.32.1 to 4.43.15 Bumps [@patternfly/react-icons](https://github.com/... 4da1836
- chore(deps): bump @redhat-cloud-services/frontend-components-translations Bumps [@redhat-cloud-services/frontend-com... 8663f6f
- chore(deps-dev): bump node-sass from 6.0.1 to 7.0.0 Bumps [node-sass](https://github.com/sass/node-sass) from 6.0.1 ... ad9c1e7
- chore(deps): bump follow-redirects from 1.14.7 to 1.14.8 Bumps [follow-redirects](https://github.com/follow-redirect... c8f0e66
- chore(deps-dev): bump @redhat-cloud-services/frontend-components-config Bumps [@redhat-cloud-services/frontend-compo... decc51c
- feat(SPM-1308): assign system to a patch set 66498d7
- feat(SPM-1306): create patch set wizard a24ccfb
- chore(deps): install sass-loader ebb07d5
- chore(release): 1.39.0 # [1.39.0](https://github.com/RedHatInsights/patchman-ui/compare/v1.38.3...v1.39.0) (2022-02-... 0d470b1
- feat(SPM-1305): create patch set table b0a411d
- chore(PatchSetWizard): rename patchSet var to rows 875fe5d
- fix(SystemsStatusReport): SPM-1358 fixed responsibility 774f57d
- fix(AdvisoriesStatusReport): SPM-1358 fixed responsiblity aa5aca9
- fix(Travis): update node version c70c540
- chore(release): 1.40.0 # [1.40.0](https://github.com/RedHatInsights/patchman-ui/compare/v1.39.0...v1.40.0) (2022-03-... 7a9de96
- chore(Travis): fix errors 306ea2c
- fix(InventoryDetail): Changed Alert to inline 05d9389
- and 492 more ...
bastilian created a review on a pull request on RedHatInsights/compliance-frontend
Sorting works again! \o/ Thank you @LightOfHeaven1994!
bastilian created a review comment on a pull request on RedHatInsights/insights-inventory-frontend
We could use the `@deprecated` tag here. Just to practice good practice. :) https://jsdoc.app/tags-deprecated
bastilian created a review on a pull request on RedHatInsights/insights-advisor-frontend
LGTM! Thank you @johnsonm325!
bastilian created a comment on a pull request on RedHatInsights/vulnerability-ui
@leSamo The dev server is not behaving correctly and I suspect it is because of the direct webpack dependencies. Basically every webpack dependency in `package.json` should be removed. We should...
bastilian created a review on a pull request on RedHatInsights/compliance-frontend
Nice work! Thank you @LightOfHeaven1994!
bastilian created a review comment on a pull request on RedHatInsights/insights-inventory-frontend
Please look into how we can get rid of the `/insights/inventory/`. We are already within `/insights/inventory/` and should not need it. If the path changes in the future we have to remember to chan...
bastilian created a review comment on a pull request on RedHatInsights/insights-inventory-frontend
> in this case most of the location would be discarded in order to arrive at the correct path I'm not sure I understand what you mean?
bastilian created a review comment on a pull request on RedHatInsights/insights-inventory-frontend
We should not have this (`/insights/inventory/` specifically) hardcoded in here. The other paths for the links are constructed with help from the pathname, would this not work for this section?
bastilian created a review comment on a pull request on RedHatInsights/insights-inventory-frontend
What's `fullTarget` for? Why do we need this?
bastilian created a review on a pull request on RedHatInsights/insights-remediations-frontend
Nice! LGTM! Thank you @leSamo!
bastilian pushed 125 commits to master bastilian/insights-remediations-frontend
- fix(SystemsTable): REMEDY-265 - Don't set columns via redux store (#339) 23eeb3a
- [REMEDY-219] Fix new loading issue To reproduce the bug: - Navigate to /insights - Click Toolkit -> Remediations in ... 7ef3b56
- Merge pull request #340 from johnsonm325/REMEDY-219-2 [REMEDY-219] Fix new loading issue b4db8f5
- chore: introduces pull request templates e67b80a
- Merge pull request #343 from mkholjuraev/pull-request-template chore: introduces pull request templates 5b75176
- fix(REMEDY-277): extract columns from reducer hash (#345) da4880e
- feat(REMEDY-235): default columns are consumed from inventory, sorting enabled (#344) 405b3b9
- fix(ActionsTable): Fixing bulk select a505832
- Merge branch 'master' into remediations-bulk-select 4600328
- Merge pull request #347 from Fewwy/remediations-bulk-select fix(ActionsTable): Fixing bulk select e703d32
- chore: introduce cypress testing (#346) * chore: introduce cypress testing * chore: move testing from travis to g... 3370969
- chore: fix git workflow test name 0a387b0
- fix(InventoryColumns): use one custom columns array to merge with inventory default columns bf9c0ee
- chore(Shared dependencies): Align versions of redux (#348) a14ff66
- Merge branch 'master' into fix-search c2e514c
- Merge pull request #350 from mkholjuraev/fix-search fix(InventoryColumns): use one custom columns array to merge wit... 5f83a80
- fix(detailsTable): bulk select fix 28fe393
- Merge branch 'master' into fixing-bulk-select 4459cc4
- fix(remediationsDetails): swapped places select none and select all bdf81be
- feat(RemediationsDetails): fixing bulk select 05362c8
- and 105 more ...
bastilian created a review on a pull request on RedHatInsights/insights-inventory-frontend
LGTM! Thank you for setting this up @gkarat!
bastilian created a comment on a pull request on RedHatInsights/insights-inventory-frontend
@AsToNlele If the links are fixed, do we still need this PR?
bastilian created a review on a pull request on RedHatInsights/insights-advisor-frontend
LGTM! Thank you @leSamo!
bastilian created a review on a pull request on RedHatInsights/compliance-frontend
Awesome! LGTM! Thank you for cleaning this up @leSamo!
bastilian created a comment on a pull request on RedHatInsights/compliance-frontend
@LightOfHeaven1994 If you need someone to look over your shoulder and help, let me know.