Closed ryangroth5 closed 3 years ago
@drewalth can't request a review from you, here's a first attempt at fixing the form.
sorry for delay on this @ryangroth5 -- I made some small changes and am testing it now. Also made two fixes:
refresh()
didn't actually re-load the photos displaying because the state hasn't updated, fixed now
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
removes VUE_APP_GRAPHQL_HTTP, not being used. replaced with VUE_APP_API_BASE_URL+"graphql"
changes photo upload type to GALLERY and post type to PHOTO_UPLOAD to get correct ID
hides upload if user is not logged in
refreshes gallery when upload is sent
uploads the fields collected in the form when it updates the post
What is the current behavior? (You can also link to an open issue here)
code indicates that the gallery is updating a POST with the id of the reach being so the wrong post is getting updated and regular users are getting a post not authorized error
code doesn't update fields from the form
code uses two ways to indicate the graphql endpoint
What is the new behavior (if this is a feature change)?
explained
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
i don't think so
Other information: addresses #1797