Skip to content

Comments

Show new fields analysis samples 2#367

Merged
aastabk merged 4 commits intosummer25-week-30from
show-new-fields-analysis-samples-2
Jul 23, 2025
Merged

Show new fields analysis samples 2#367
aastabk merged 4 commits intosummer25-week-30from
show-new-fields-analysis-samples-2

Conversation

@aastabk
Copy link
Contributor

@aastabk aastabk commented Jul 23, 2025

No description provided.

@aastabk aastabk requested review from mortenlyn and omfj July 23, 2025 06:40
@aastabk aastabk self-assigned this Jul 23, 2025
marker = models.ForeignKey(f"{an}.Marker", on_delete=models.PROTECT)
transaction = models.UUIDField(blank=True, null=True)

# NEW FIELDS for status tracking
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
# NEW FIELDS for status tracking
# Fields for status tracking

It won't be "new fields" forever.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed.

@aastabk aastabk requested a review from mortenlyn July 23, 2025 07:37
@aastabk aastabk force-pushed the show-new-fields-analysis-samples-2 branch from 1273d7f to 8878b8b Compare July 23, 2025 08:31
@aastabk aastabk merged commit 57aad3a into summer25-week-30 Jul 23, 2025
8 checks passed
@aastabk aastabk deleted the show-new-fields-analysis-samples-2 branch July 23, 2025 08:32
omfj pushed a commit that referenced this pull request Jul 24, 2025
* Added PCR, analysis and output field to samples

* Added PCR, analysis and output for analysis samples.

* Fix linter error

* Changed comment
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.

3 participants