Skip to content

[BI-1315] germplasm import: fix next val germplasm sequence call#139

Merged
ctucker3 merged 1 commit intodevelopfrom
bug/BI-1315
Jan 26, 2022
Merged

[BI-1315] germplasm import: fix next val germplasm sequence call#139
ctucker3 merged 1 commit intodevelopfrom
bug/BI-1315

Conversation

@ctucker3
Copy link
Contributor

@ctucker3 ctucker3 commented Jan 26, 2022

Description

There is a bug with dsl.nextval() that gets our germplasm sequence values where it surrounds program keys that aren't lowercased with quotes, select nextval('"program_sequence"'). This causes a not found error in psql.

Dependencies

none

Testing

Uppercase the program key in one of your germplasm sequences. For example, change bb_germplasm_sequence to BB_germplasm_sequence.

Checklist:

  • I have performed a self-review of my own code
  • I have tested my code and ensured it meets the acceptance criteria of the story
  • I have create/modified unit tests to cover this change
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to documentation
  • I have run TAF.

@github-actions github-actions bot added the bug Something isn't working label Jan 26, 2022
@ctucker3 ctucker3 merged commit 28484ba into develop Jan 26, 2022
@ctucker3 ctucker3 deleted the bug/BI-1315 branch January 26, 2022 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants