Add update functionality to rules within schedule page and pre-populate information - Githubissues

UA-Smart-Signage-Platform / Content-Manager-System-and-Content-Creator-Tool

Content Manager System and Content Creator Tool developed to facilitate the presentation of information.

GNU General Public License v3.0

1 stars 0 forks source link

Add update functionality to rules within schedule page and pre-populate information #143

Closed pdPinho closed 5 months ago

pdPinho commented 5 months ago

Issue

Closes #139

Reason for this change

Previously it was possible to update the rule's options besides its priority. Now we are capable of changing every field within it.

Description of changes

Added functionality to update each rule in particular
Added fetch to get information about selected rule
Made small change to backend's TemplateGroupService to accept different widgets
Fixed critical group deletion bug that prevented user from creating rules
Fixed critical run condition bug when loading schedule templates
Fixed small bug when sending information about non-Media widgets

Description of how you validated changes

Played around with schedule, groups and alike. Nothing broke... so far.

Checklist

[X] I have tested my changes locally.
[ ] I have updated the documentation to reflect my changes.
[ ] I have added/updated unit tests (if applicable).
[X] My code adheres to the CONTRIBUTING GUIDE

Aditional Information

N/A

sonarcloud[bot] commented 5 months ago

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

pdPinho commented 5 months ago

lgtm, except the part of +1 -1

... I don't really know why it doesn't work without that :sweat_smile:

sonarcloud[bot] commented 5 months ago

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud