Skip to content
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.

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

FEMS-Backports 2024-08 (2) #2771

Merged
merged 17 commits into from
Aug 31, 2024
Merged

FEMS-Backports 2024-08 (2) #2771

merged 17 commits into from
Aug 31, 2024

Conversation

sfeilmeier
Copy link
Contributor

No description provided.

sfeilmeier and others added 17 commits August 30, 2024 23:16
Wrong variable used in consumption detail html

Reviewed-by: Stefan Feilmeier <[email protected]>
Co-authored-by: Lukas Rieger <[email protected]>
Co-committed-by: Lukas Rieger <[email protected]>
- Introduction of [Routing Guards](https://v17.angular.io/api/router#types)
- Not visible routes could be accessed through manually going there
- Adding edge role check before accessing this route

Reviewed-by: Sagar Venu <[email protected]>
Center refresh-content-spinner

Reviewed-by: Sagar Venu <[email protected]>
- fix start for timeranges with not the first day of the month
- fix readonly only worked with debugmode

Reviewed-by: Hueseyin Sahutoglu <[email protected]>
Co-authored-by: Michael Grill <[email protected]>
Co-committed-by: Michael Grill <[email protected]>
Update Angular, Zone.js, Typescript

Co-authored-by: Kai Jeschek <[email protected]>
Reviewed-by: Sagar Venu <[email protected]>
- Enforce usage of double quotes
- prohibit single quotes
- backticks are still usable -> [ref](https://eslint.org/docs/latest/rules/quotes#double)

Reviewed-by: Sagar Venu <[email protected]>
- Add eslint rule [`switch-exhaustiveness-check`](https://typescript-eslint.io/rules/switch-exhaustiveness-check/)

Reviewed-by: Sagar Venu <[email protected]>
- Updating android api level due to googles new policy, enforcing `ANDROID_API_LEVEL=34` to be minimum version

Reviewed-by: Kai Jeschek <[email protected]>
Add CI pipeline for building Android apps

Co-authored-by: Lukas Rieger <[email protected]>
Reviewed-by: Lukas Rieger <[email protected]>
Reviewed-by: Sebastian Asen <[email protected]>
Co-authored-by: Johann Kaufmann <[email protected]>
Co-committed-by: Johann Kaufmann <[email protected]>
If a history chart takes too long to load and the user switches the timerange,
the old timerange will be shown instead of the actual one (at least for a few seconds, until the new chart is finished)

Co-authored-by: Lukas Rieger <[email protected]>
Reviewed-by: Lukas Rieger <[email protected]>
Co-authored-by: Fabian Brandtner <[email protected]>
Co-committed-by: Fabian Brandtner <[email protected]>
- Replace `setCurrentComponent` with `getCurrentEdge`, setCurrentComponent is already deprecated and shouldnt be used anymore

Reviewed-by: Sagar Venu <[email protected]>
- Unify and refactor all FixDigitalOutputControllers and SingleThresholds in one widget in history
- Refactoring them and using the new `CumulatedLevelActiveTime`-Channel

Co-authored-by: Sebastian Asen <[email protected]>
Reviewed-by: Sebastian Asen <[email protected]>
- Fix `TypeError: this.options.plugins is undefined`
- To reproduce the error see "live" screen and open the modal for Time-of-Use with ADMIN role.

Reviewed-by: Sagar Venu <[email protected]>
Copy link

codecov bot commented Aug 31, 2024

Codecov Report

Attention: Patch coverage is 50.89286% with 110 lines in your changes missing coverage. Please review.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #2771      +/-   ##
=============================================
+ Coverage      55.88%   55.91%   +0.04%     
- Complexity      8044     8052       +8     
=============================================
  Files           2077     2079       +2     
  Lines          87948    87949       +1     
  Branches        6494     6499       +5     
=============================================
+ Hits           49142    49170      +28     
+ Misses         37122    37088      -34     
- Partials        1684     1691       +7     

@sfeilmeier sfeilmeier merged commit 4a852f7 into develop Aug 31, 2024
3 of 4 checks passed
@sfeilmeier sfeilmeier deleted the feature/fems-backport branch August 31, 2024 12:51
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.

1 participant