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

GT-95 Softcoding of service type production<=>monitored service flag relations. #209

Merged
merged 3 commits into from Mar 31, 2023
Merged

Conversation

ghost
Copy link

@ghost ghost commented Mar 9, 2020

Draft pull request while doctrine schema update issues are checked and resolved.

Add monitoring-exception-allowed flag to service type table.
Associated GUI changes and cleanups.
Add instance count column to service type overview table.
Add related unit tests.

Closes #156

@ghost
Copy link
Author

ghost commented Mar 12, 2020

I was unable to replicate the schema update problem I ran into originally, so have reverted to suggested definition - which was what I first had afaiks :-(

@ghost ghost marked this pull request as ready for review March 12, 2020 11:59
@gregcorbett gregcorbett self-assigned this Mar 19, 2020
@gregcorbett gregcorbett added this to the 5.8.0 milestone Mar 19, 2020
@gregcorbett gregcorbett added the schema change Issues or PRs that require a schema change to implement. label May 27, 2021
@gregcorbett gregcorbett modified the milestones: 5.8.0, 5.10.0 May 27, 2021
@gregcorbett gregcorbett modified the milestones: 5.10.0, August 2022 Jul 11, 2022
@gregcorbett gregcorbett modified the milestones: August 2022, November 2022 Sep 6, 2022
@gregcorbett gregcorbett modified the milestones: November 2022, June 2023 Nov 2, 2022
Copy link
Member

@gregcorbett gregcorbett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one comment so far. Could you also rebase to resolve the conflicts?

@tofu-rocketry tofu-rocketry changed the title Softcoding of service type production<=>monitored service flag relations. GT-95 Softcoding of service type production<=>monitored service flag relations. Mar 7, 2023
@ghost ghost self-requested a review as a code owner March 20, 2023 16:03
gregcorbett
gregcorbett previously approved these changes Mar 24, 2023
@gregcorbett
Copy link
Member

rebasing on top of latest dev

Ian added 2 commits March 24, 2023 10:55
Add monitoring-exception-allowed flag to service type table,
associated GUI changes and cleanups.
Add instance count column to service type overview table.
- If you line break in a string, the whitespace is taken
  literally
- as such, the whitespace was appearing in the link to the
  ServiceType
- the link still worked, but it seemed slow to respond.
@gregcorbett gregcorbett dismissed their stale review March 31, 2023 09:52

going to do some final testing before merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement schema change Issues or PRs that require a schema change to implement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

'production => monitored' rule should be a property of the ServiceType in the database and not hardcoded.
3 participants