Skip to content
This repository has been archived by the owner on Apr 23, 2024. It is now read-only.
/ js-client Public archive

fix(#336): Can't fetch a template just after creation. #337

Merged
merged 14 commits into from
Jun 20, 2022
Merged

Conversation

iagojsilva
Copy link
Contributor

Adresses: #336

Important to say that, the issue was even bigger than what was described on gitea, so when any template was created, an 400 request was received and didn't go to /templates page after clicking on the save button. (And probably that's why drone was failing on template tests)

issue5082cantAddAnyTemplate.mp4

With the current solution, beyond the fact, that the template is working, and the drone will probably be fixed, now we do one request (create one template) to the back-end instead of two (create one template and get one template).

After the implementation, here is the behavior on the template page (add, edit and delete works correctly):

showingFix336.mp4

iagojsilva and others added 3 commits June 15, 2022 18:58
Squashed commit of the following:

commit 22e5f8f
Author: GeraldHamiltonWicks <[email protected]>
Date:   Wed Jun 15 16:52:56 2022 -0300

    removing useless import

commit 7a00017
Author: GeraldHamiltonWicks <[email protected]>
Date:   Wed Jun 15 16:50:37 2022 -0300

    adding spaces

commit 9d61747
Author: GeraldHamiltonWicks <[email protected]>
Date:   Wed Jun 15 16:48:39 2022 -0300

    removing logs

commit 2e6b530
Author: GeraldHamiltonWicks <[email protected]>
Date:   Wed Jun 15 16:47:10 2022 -0300

    fixing createOneTemplate
@iagojsilva iagojsilva self-assigned this Jun 16, 2022
@iagojsilva iagojsilva merged commit 995c94e into dev Jun 20, 2022
@michael-wisely-gravwell michael-wisely-gravwell deleted the fix/templates branch July 22, 2022 20:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants