Skip to content

Generalize source object #61

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 27 commits into from
Oct 16, 2025
Merged

Generalize source object #61

merged 27 commits into from
Oct 16, 2025

Conversation

btry
Copy link
Collaborator

@btry btry commented Sep 19, 2025 •
edited
Loading

  • Rename from CarbonIntensitySource to Source to manage sources related to other aspects of LCA.
  • remove location / zone association by country and state, use the source_zone relation table instead.

@btry btry changed the base branch from develop to support/1.1.0 September 19, 2025 11:47
@btry btry changed the base branch from support/1.1.0 to develop September 19, 2025 11:48
@btry btry force-pushed the refactor/generalize_source branch from 82979d1 to f871f0c Compare September 19, 2025 12:07
@btry btry changed the base branch from develop to support/1.1.0 September 19, 2025 12:09
@btry btry changed the base branch from support/1.1.0 to develop September 19, 2025 12:09
@btry btry force-pushed the refactor/generalize_source branch 2 times, most recently from a13dfe9 to 9c87e79 Compare September 19, 2025 13:03
@btry btry force-pushed the refactor/generalize_source branch 2 times, most recently from 4cea583 to 2a7edb5 Compare October 3, 2025 16:46
@btry btry marked this pull request as draft October 3, 2025 16:48
@btry btry force-pushed the refactor/generalize_source branch 2 times, most recently from 28ca384 to 250d85c Compare October 6, 2025 07:59
@btry btry mentioned this pull request Oct 9, 2025
Closed
@btry btry force-pushed the refactor/generalize_source branch 10 times, most recently from 6dda41c to 2faf23e Compare October 15, 2025 07:55
btry added 20 commits October 16, 2025 13:51
@btry
fix(Impact\History\AbstractAsset): test memory before calculating a c…
0c47508 
…arbon emission

A result was calculated but not saved, wasting computational resources
@btry
test: replace deprecated methods
61cf89f
@btry
refactor(Zone): set alias to location class
519bb7e
@btry
feat(Zone): update algorithms to find zone from location or asset
a753af5
@btry
test(Install): add message on failure
59e8f2c
@btry
tst(Location): fix test method name
7802940
@btry
test(Impact\History\Monitor): update out of date itemtype
bb444d7
@btry
test: update tests with location /zone relation
1efb062
@btry
feat(Location): associate location to a carbon intensity zone
1981acf
@btry
fix(Install): avoid use of non-existing classes in migration code
bd8132f
@btry
style: php lint
7ed0dce
@btry
style(Zone): remove unused var
36249c6
@btry
style(Location): twig code style
cffab41
@btry
feat(Location): update data completion diagnosis
1b4162f
@btry
fix(Install): reoder upgrade steps
ab71f30
@btry
feat(Location): remove location / carbon intensity relation based on …
55a12e3 
…country or state
@btry
feat(Install): migrate location / zone relation
272952e
@btry
refactor(Source_Zone): code cleanup
07288ca
@btry
fix(Dashboard\Widget): fix bad argument when computing Y scale
414edd6
@btry
feat(SearchOption): update search option for historizable status
c2a4d3e
@btry btry force-pushed the refactor/generalize_source branch from 39cf187 to c2a4d3e Compare October 16, 2025 11:51
@btry btry marked this pull request as ready for review October 16, 2025 12:22
@btry btry merged commit 2a13de4 into pluginsGLPI:develop Oct 16, 2025
3 checks passed
@btry btry deleted the refactor/generalize_source branch October 16, 2025 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@btry