-
Notifications
You must be signed in to change notification settings - Fork 4
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
Implement Poet Upload & Persistence #14
Labels
enhancement
New feature or request
Comments
asmr-hex
pushed a commit
that referenced
this issue
Feb 22, 2018
poet files on host fs, included sample data dir with sample program/parameters files, added constants for file upload API, filled in the body of CreatPoet in API handlers, created Users and Poets Tables on Platform setup, added Language field/column to Poets data model TODOS: * need to return HTTP responses to users in CreatePoet handler * need to write unit tests for Creat Poet handler * Must implement User creation/auth/session stuff (idk???) in order to properly create a poet (since the poets table in Postgres references UUIDs within the users table) * possibly refactor the id arg in poet.Create.
asmr-hex
pushed a commit
that referenced
this issue
Feb 22, 2018
asmr-hex
pushed a commit
that referenced
this issue
Feb 22, 2018
in-memory session token cache, modified user table to include hashed password and salt, implement session methods for correlating userIds to session tokens and vice versa ❀‿❀ -- yay!! (づ。◕‿‿◕。)づ -- wheeeeee ╰(◡‿◡✿╰) -- whoooooo TODOS: * use CookieJar instead of Cookie? * standardize form vs. data for requests * implement session token expiration policy goroutine * actually send responses to clients (send session token cookie as well) * write unit test for all of this ;___;
asmr-hex
pushed a commit
that referenced
this issue
Feb 23, 2018
Major Changes: * add http responses for login, register, and create poet * send session token in login response * implemement session token background sweeper Notes: * tests are currently broken (1 server test) * token timeout duration is hard-coded, move to config ( ˘ ³˘)♥ ( ˘ ³˘)♥ ( ˘ ³˘)♥ ( ˘ ³˘)♥ ( ˘ ³˘)♥
asmr-hex
pushed a commit
that referenced
this issue
Feb 24, 2018
asmr-hex
pushed a commit
that referenced
this issue
Feb 25, 2018
Major Changes: * add create, read, update, delete validation for poets * implement name regexp validator (supports unicode) * unit tests for name regexp validator TODO: * validate that poet can only be created/modified/deleted by designer * test coverage for poet validator function
Merged
asmr-hex
pushed a commit
that referenced
this issue
Mar 1, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Overview
when creating poets, a user should be able to upload an executable file and an optional parameters file
Requirements
TODO
unit tests (for everything we did)(delegated to issue Increase Test Coverage -____- #16)refactor the(delegated to issues refactors #17)User.Create()
id argument weirdnessstandardize the http data format (form vs. raw data)(delegated to issue Server API support Data & Form-Data #18)The text was updated successfully, but these errors were encountered: