Skip to content

replace report_issue and resolve_issue endpoints with new "issue_handling" endpoint#415

Open
Abh1ramManugur1 wants to merge 2 commits into
scimma:mainfrom
Abh1ramManugur1:413-issue-handling
Open

replace report_issue and resolve_issue endpoints with new "issue_handling" endpoint#415
Abh1ramManugur1 wants to merge 2 commits into
scimma:mainfrom
Abh1ramManugur1:413-issue-handling

Conversation

@Abh1ramManugur1

Copy link
Copy Markdown
Collaborator

Fixes #413.

Description of the Change

functionality unchanged, endpoint logic condensed into single view function

Checklist

Please check all that apply to your proposed changes

  • HTML code change
  • Added package dependency
  • Added data
  • Changed django model
  • Documentation change
  • Added or changed TaskRunner

Additional context

functionality unchanged, endpoint logic condensed into single view function
@manning-ncsa

Copy link
Copy Markdown
Collaborator

@Abh1ramManugur1 a few points:

  • Did you interactively test this version on a result page?
  • Also, did you run the unit tests (i.e. bash run/blastctl ci up)?
  • Please update the Changelog with a line describing the changes. Don't worry about changing the version number yet. We should probably consolidate these API changes into a single 2.0.0 release.

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.

Refactor issue endpoints into single endpoint with added action parameter

2 participants