Skip to content
This repository was archived by the owner on Jul 7, 2021. It is now read-only.

move api over to gulag #62

Open
Airkek opened this issue Feb 28, 2021 · 6 comments · Fixed by #63
Open

move api over to gulag #62

Airkek opened this issue Feb 28, 2021 · 6 comments · Fixed by #63
Assignees
@yo-ru @varkaria
Labels
help wanted Extra attention is needed postponed This will not be worked on until later

Comments

@Airkek
Copy link
Contributor

Airkek commented Feb 28, 2021

cmyui changed sql scheme in v3.2.2 and it's broke api/get_user

image
@Airkek Airkek added bug Something isn't working help wanted Extra attention is needed labels Feb 28, 2021
@alowave223
Copy link
Contributor

image

@alowave223
Copy link
Contributor

and the api from the web side is no longer relevant because it was added to gulag
image

@cmyui
Copy link
Collaborator

cmyui commented Feb 28, 2021 •
edited
Loading

Will be moving all this api stuff to gulag soon, have already started work on it.

@yo-ru
Copy link
Owner

yo-ru commented Mar 1, 2021

I'll push a hotfix for /api/get_user.

I wont move gulag-web over to gulag's internal api until it is fully finished.

@yo-ru yo-ru linked a pull request Mar 1, 2021 that will close this issue
hotfix for sql change in latest gulag #63
Merged
@yo-ru yo-ru closed this as completed in #63 Mar 1, 2021
@yo-ru yo-ru added the postponed This will not be worked on until later label Mar 3, 2021
@yo-ru yo-ru changed the title api/get_user doesn't work with gulag v3.2.2 move api over to gulag Mar 3, 2021
@yo-ru yo-ru removed the bug Something isn't working label Mar 3, 2021
@yo-ru
Copy link
Owner

yo-ru commented Mar 3, 2021

Will reopen and close upon merge of api change

@yo-ru yo-ru reopened this Mar 3, 2021
@cmyui
Copy link
Collaborator

cmyui commented Mar 11, 2021

Note that most of the API is now complete under the devel branch of gulag, feel free to move to the new handlers & remove the old api blueprint here.

If there's any more handlers needed, feel free to leave them here.

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

@yo-ru yo-ru

@varkaria varkaria

Labels
help wanted Extra attention is needed postponed This will not be worked on until later
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

hotfix for sql change in latest gulag yo-ru/gulag-web
5 participants
@cmyui @yo-ru @Airkek @alowave223 @varkaria