Skip to content

Add new report for EOL Assets#11296

Open
veenone wants to merge 7 commits intogrokability:developfrom
veenone:feat/eol_report
Open

Add new report for EOL Assets#11296
veenone wants to merge 7 commits intogrokability:developfrom
veenone:feat/eol_report

Conversation

@veenone
Copy link
Copy Markdown
Contributor

@veenone veenone commented Jun 12, 2022

Description

This PR adds new reporting especially for EOL assets information.
The table will only display assets with filled purchase date, and models has the EOL period set

Fixes #10687 -> just to display, no notification

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

  • Test A - adding assets with complete data (EOL period on model, and purchase date)
  • Test B - adding assets with incomplete data (missing EOL period on model, missing purchase date)
  • Test C - test report generation

Test Configuration:

  • PHP version: 7.4.27
  • MySQL version:
  • Webserver version:
  • OS version: Windows 10

Checklist:

Preview

  • Shortcut menu
    image
  • Report menu
    image

@veenone veenone requested a review from snipe as a code owner June 12, 2022 14:23
@veenone veenone changed the title [Feature] - Report for EOL Assets [Feature] - Add new report for EOL Assets Jun 12, 2022
@Christopheric1
Copy link
Copy Markdown

This looks good and will be very helpful! Thank you.

@veenone
Copy link
Copy Markdown
Contributor Author

veenone commented Jun 30, 2022

all done

@veenone veenone changed the title [Feature] - Add new report for EOL Assets Add new report for EOL Assets Jul 27, 2022
@Asyraf777
Copy link
Copy Markdown

Hi how to import this feature ya?for the code just add in into the file?

@veenone
Copy link
Copy Markdown
Contributor Author

veenone commented Nov 13, 2022

Hi how to import this feature ya?for the code just add in into the file?

sorry it's been a long time since my last time checking my PRs.
it's either waiting for this PR to be merged by the owners, or download the codes in this PR and integrate it yourself in your environment. just make sure you resolve any conflicts if any

@RobertAzelis
Copy link
Copy Markdown

Hi, maybe better to add it as filter on Assets list, like it's in Software licences
image

@snipe snipe self-assigned this Oct 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants