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

Feat: 343 activity ng transmission non compression #2772

Merged
merged 14 commits into from
Feb 5, 2025

Conversation

pbastia
Copy link
Contributor

@pbastia pbastia commented Jan 31, 2025

No description provided.

@pbastia pbastia marked this pull request as draft January 31, 2025 18:09
@pbastia pbastia force-pushed the feat/343-activity-ng-transmission-non-compression branch from bce72e2 to fbc6ca3 Compare February 3, 2025 17:00
@pbastia pbastia marked this pull request as ready for review February 3, 2025 18:10
@pbastia pbastia force-pushed the feat/343-activity-ng-transmission-non-compression branch from 6934d91 to af900e7 Compare February 4, 2025 23:25
Copy link
Contributor

@dleard dleard left a comment

Choose a reason for hiding this comment

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

Love the tests/uiSchema stuff! One small change needed to a source type & this is good to go.

@@ -0,0 +1,88 @@
{
"type": "object",
"title": "Flaring stacks",
Copy link
Contributor

Choose a reason for hiding this comment

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

We have two very similar source types in our db: "Flaring stacks" & "Flare stacks". I remember asking IRC about this and they said that they are pretty much the same thing, but there is a slight difference (or they are at least named differently in the regs).

Suggested change
"title": "Flaring stacks",
"title": "Flare stacks",

True,
),
("5_blowdown_venting", "Blowdown venting", False, False),
("6_flare_stacks", "Flaring stacks", True, True),
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
("6_flare_stacks", "Flaring stacks", True, True),
("6_flare_stacks", "Flare stacks", True, True),

ConfigurationElement.objects.bulk_create(
ConfigurationElement(
activity=ng_non_compression_activity,
source_type=SourceType.objects.get(name="Flaring stacks"),
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
source_type=SourceType.objects.get(name="Flaring stacks"),
source_type=SourceType.objects.get(name="Flare stacks"),

@pbastia pbastia merged commit 1c7989a into develop Feb 5, 2025
42 checks passed
@pbastia pbastia deleted the feat/343-activity-ng-transmission-non-compression branch February 5, 2025 00:55
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