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.

Sign up for GitHub

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

Jump to bottom

testplans/views/CaseActions needs to support GET and POST or be refactored #196 #21

Closed
atodorov opened this issue May 26, 2017 · 1 comment
Closed

testplans/views/CaseActions needs to support GET and POST or be refactored #196 #21

atodorov opened this issue May 26, 2017 · 1 comment
Labels
refactoring

Comments

@atodorov
Copy link
Member

From original discussion at #176

This is an inner class, defined inside the cases method, which allows both GET and POST. Either delete_cases needs to require only POST or you need to take both of them into account.

NOTE: this inner class is also similar to the ones discussed in #184
@atodorov
Copy link
Member Author

See Nitrate/Nitrate#255

atodorov pushed a commit that referenced this issue Feb 28, 2018
@tkdchen @atodorov
Refactor inner action class CaseActions. Fixes #21
a9d5157 
CaseActions is moved and separated to individual view classes for each
operation request. Original cases view method is not used anymore and
removed.

Signed-off-by: Chenxiong Qi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
refactoring
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@atodorov