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.

Sign up for GitHub

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

Jump to bottom

When updating Site Visit, Instrument output variable not shown in summary for Instrument calibration #283

Closed
Elijahwalkerwest opened this issue Apr 6, 2018 · 4 comments
Closed

When updating Site Visit, Instrument output variable not shown in summary for Instrument calibration #283

Elijahwalkerwest opened this issue Apr 6, 2018 · 4 comments
Assignees
@Elijahwalkerwest
Labels
bug fixed ready for testing

Comments

@Elijahwalkerwest
Copy link

When you have an Instrument Calibration, it won't show the IOV in the summary page of a site visit.
@Elijahwalkerwest
Copy link
Author

Sorry, to clarify, when creating or updating a site visit that includes an instrument calibration action.

@Elijahwalkerwest Elijahwalkerwest self-assigned this Apr 6, 2018
Elijahwalkerwest added a commit that referenced this issue Apr 6, 2018
@Elijahwalkerwest
Fix for #283, #279, and fix for the results and equipment data manage…
5712529 
…ment in #282
Elijahwalkerwest added a commit that referenced this issue Apr 6, 2018
@Elijahwalkerwest
Merge pull request #284 from UCHIC/django-elijah-dev
fec114f 
Fix for #283, #279, and fix for the results and equipment data manage…
@AmberSJones
Copy link

Is this what you are referring to? The variable is displayed when editing, but not on the Calibration summary view.

image

image

@Elijahwalkerwest
Copy link
Author

In the calibration details as well as the summary after you finish editing. About to commit and push a fix for both.

Elijahwalkerwest added a commit that referenced this issue Apr 16, 2018
@Elijahwalkerwest
Fixes for #283, #286, #270
bbe06d5
Elijahwalkerwest added a commit that referenced this issue Apr 24, 2018
@Elijahwalkerwest
Merge pull request #287 from UCHIC/django-elijah-dev
0e6f855 
Fixes for #283, #286, #270
@AmberSJones
Copy link

This appears to be addressed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Elijahwalkerwest Elijahwalkerwest

Labels
bug fixed ready for testing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@AmberSJones @Elijahwalkerwest