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

(V2 post-mvp) Drag and Drop: Add 'position' column #264

Open
ethoreson opened this issue Nov 25, 2019 · 1 comment
Open

(V2 post-mvp) Drag and Drop: Add 'position' column #264

ethoreson opened this issue Nov 25, 2019 · 1 comment
Labels
Projects
Milestone

Comments

@ethoreson
Copy link
Collaborator

4 parts, this issue will need to be split later:

  1. Add an INT column to PrintJobs table: 'position'. Only printjobs with a queueing status can be rearranged, otherwise 'position' will be null & drag/drop not enabled on frontend. INTs will be 10 #s off from the last created (see Create Endpoints for Analysis View #4).
  2. GET request, /api/print-jobs/{printId}/position - When a queued item is moved up or down, it will get the position number of its new ceiling and floor neighbors, and its new position will split the difference between those INTs.
  3. Update the PUT /api/print-jobs/{printId} request to include 'position' as another valid attribute change.
  4. When a new card is created, its position should be +10 of the highest current position.

UNSURE:

  • What happens 6 months down the road, should we reset positions to restart at 0 at a certain point?
  • Would changing from INT to DECIMAL be overkill, or necessary?
@ethoreson ethoreson created this issue from a note in ideaLab (To do) Nov 25, 2019
@hannahkamundson hannahkamundson added this to the Post-MVP milestone Dec 3, 2019
@kaansonmezoz
Copy link
Collaborator

Any updates about "UNSURE" parts ?

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

No branches or pull requests

3 participants