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

Update answer attachment within create function #2736

Closed
wants to merge 1 commit into from

Conversation

reinbach
Copy link
Contributor

@reinbach reinbach commented Jul 1, 2020

No description provided.

@robertavram
Copy link
Member

@reinbach any ticket relating to this? do you remember what the issue was here?

@reinbach
Copy link
Contributor Author

reinbach commented Oct 6, 2020

@reinbach any ticket relating to this? do you remember what the issue was here?

This was me attempting to fix an issue that was happening in prod. I think we can close this PR, as we want with requiring the FE to make another request. @ntrncic thoughts?

@robertavram
Copy link
Member

@reinbach there's an issue with attachments that is being reported in prod.. but I have no idea what it is.. do you remember the issue that we're trying to fix here?

@reinbach
Copy link
Contributor Author

reinbach commented Oct 6, 2020

@robertavram the issue was that the response from the BE did not have the updated data for the attachment. But a refresh did.
This hack was an attempt to make sure the first response did have the updated data.

This issue is not happening locally (or in dev, I believe) just in prod

@robertavram
Copy link
Member

robertavram commented Oct 6, 2020 via email

@robertavram
Copy link
Member

nope

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants