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

Create release endpoint #19

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Dmitry221060
Copy link

@Dmitry221060 Dmitry221060 commented Sep 24, 2023

Closes #18

I've never worked with php before and wasn't able to get project running(it always responds with ERR_CONNECTION_REFUSED) so there are few things I need help with:
1. I left 3 TODO comments on parts of code that I'm not sure are correct.
2. Since I wasn't able to run it, I can't test it either.
3. New post method uses web session token for authentication, but this token only lives for 2 weeks. We want to add a page/endpoint that allows to generate an API key(just like session token on login) for a user that won't expire.

@Dmitry221060 Dmitry221060 changed the title Create release waypoint Create release endpoint Sep 25, 2023
api.php Outdated Show resolved Hide resolved
api.php Outdated Show resolved Hide resolved
api.php Outdated Show resolved Hide resolved
@tyronx
Copy link
Contributor

tyronx commented Feb 18, 2024

any chance you could test this still?

@Dmitry221060
Copy link
Author

Hello, I've tried to set it up once again, but neither docker nor Remote Containers work for me. I can't get the website running even on the master branch and therefore can't test this PR.
I'm ok with it being closed unless someone wants to take over.

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

Successfully merging this pull request may close these issues.

[Feature request] API endpoint for CD to upload new versions of the mod
2 participants