Skip to content

Conversation

@mtelahun
Copy link
Contributor

Migrate itm to 18.0

mtelahun and others added 30 commits April 10, 2025 14:22
Generated random password was being doubly encrypted. First by
the get_random_password() method itself, and a second time
when the already encrypted url-safe string was written to
the database.

Fixes: trevi-software#9
Adds an icon next to password field to toggle plaintext password.

Fixes: trevi-software#11
…sets

Components are discreent parts of an IT asset that may be obtained
separately but need to be tracked none-the-less. A Component
includes: Processors, RAM, Expansion cards, controllers, etc.
It does not include, for example, a monitor (which is an asset
in its own right). A starter list of component data comes
pre-populated already.
Trying to create a worklog from the equipment form view caused an
exception because of self.env.user was misspelled self.env.user_id.

Fixes trevi-software#17
Added a help message that will appears when user is hovering mouse onto a field. Addition consist of :
name field
description field
partner field
function_ids field
os_name field
site field
Added a help message that will appears when user is hovering mouse onto a field. Addition consist of :
equipment_type selection
is_backup bool
is_application bool
is_contracted bool
is_partitioned bool
is_config_file bool
ad_service_id selection
dhcp_service_id selection
function_host bool
function_router bool
function_fileserver bool
function_database bool
mtelahun and others added 20 commits April 10, 2025 14:22
… show them in equip. list view

Current dhcp server usually assign the same IP to a device, so
keeping a record of it is useful. I find this especially usefull
for printers. In the future I would like to hook up a netdiscover
type tool that will keep this ip updated.
Also, it can be useful to see a device's assigned
IP addresses straight from the list view instead of digging into
to form view.
Now we can have the same IP address in multiple networks.
@mtelahun mtelahun force-pushed the 18.0-mig-itm branch 6 times, most recently from 7446b9e to 874303c Compare April 11, 2025 16:27
@codecov
Copy link

codecov bot commented Apr 11, 2025

Codecov Report

Attention: Patch coverage is 74.81957% with 314 lines in your changes missing coverage. Please review.

Please upload report for BASE (18.0@5c0927d). Learn more about missing BASE report.

Files with missing lines Patch % Lines
itm/models/service_ad.py 37.25% 96 Missing ⚠️
itm/models/equipment.py 70.00% 50 Missing and 7 partials ⚠️
itm/models/access.py 65.00% 47 Missing and 9 partials ⚠️
itm/models/equipment_component.py 67.74% 37 Missing and 3 partials ⚠️
itm/wizard/create_credential.py 39.62% 32 Missing ⚠️
itm/models/site.py 84.74% 7 Missing and 2 partials ⚠️
itm/models/service_wireless.py 73.07% 7 Missing ⚠️
itm/models/partner.py 72.72% 6 Missing ⚠️
itm/models/equipment_mapping.py 82.14% 5 Missing ⚠️
itm/tests/test_component.py 95.34% 2 Missing ⚠️
... and 2 more
Additional details and impacted files
@@           Coverage Diff           @@
##             18.0      #57   +/-   ##
=======================================
  Coverage        ?   74.81%           
=======================================
  Files           ?       35           
  Lines           ?     1247           
  Branches        ?      143           
=======================================
  Hits            ?      933           
  Misses          ?      291           
  Partials        ?       23           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mtelahun mtelahun merged commit bd1643e into trevi-software:18.0 Apr 14, 2025
5 of 6 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