Skip to content

refactor/fields#41

Open
tiredkangaroo wants to merge 5 commits intomainfrom
refactor/fields
Open

refactor/fields#41
tiredkangaroo wants to merge 5 commits intomainfrom
refactor/fields

Conversation

@tiredkangaroo
Copy link
Copy Markdown
Collaborator

new fields system

  • fixes race condition
  • populated schema goes through the HandlerFunc
  • uses generics to avoid type assertions

fields.go

  • cleaned up dead code
  • clean reflection (easy to understand)
  • better organized (clarity)
  • added tests (fields_test.go)

rewrote some unintelligent code

  • some bad code practices and some just bad code: fixed up

todo for this PR

  • rename puff.Route.Protocol -> puff.Route.Method (clarity purposes, not sure if it should be in this pr though)
  • add validation for T in puff method functions (puff.Get, puff.Post, puff.Put, etc.)

…dropdown empty (correct behavior would be populated with router name) not working
…dropdown empty (correct behavior would be populated with router name) not working
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.

1 participant