Skip to content
Snippets Groups Projects

Confirmed project shifts

Merged
Richrequested to merge
confirmed-project-shifts into master
1 open thread

This is a working version with a method call to retrieve project staffing information. It works but I need to do more testing around things like the user being signed in as an admin/lead as they may get the data twice.

Edited by Rich

Merge request reports

Merged by pietropietro Mar 19, 2018 (Mar 19, 2018 8:47am UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
19 type: Number
20 label: () -> i18n.__("abate:volunteers","min_people")
21 optional: true
22 autoform:
23 afFieldInput:
24 min: 1
25 placeholder: "min"
26 max:
27 type: Number
28 label: () -> i18n.__("abate:volunteers","max_people")
29 optional: true
30 # TODO: if max is not set, it should be equal to min
31 autoform:
32 afFieldInput:
33 placeholder: "max"
34 signedUp:
  • Author Maintainer

    Hmm, I was wondering if we'd need to do this. Could be messy, but might be better than querying multiple collections all the time. The problem with using a non-relational database for relational data...

  • I'm trying to write some aggregation queries, but it the version of mongodb with metero 1.6 is 3.2 and aggregations start to be useful from 3.4 ... and now the latest release is 3.6. so I'm struggling a bit.

  • Please register or sign in to reply
  • Rich added 2 commits

    added 2 commits

    • 892e8259 - Set confirmed days correctly for projects on lead page
    • 54a1d39b - Tweak project graph to use inclusive dates

    Compare with previous version

  • Rich added 1 commit

    added 1 commit

    • 406e7e43 - Remove requirement to set times for a project

    Compare with previous version

  • Rich unmarked as a Work In Progress

    unmarked as a Work In Progress

    • pietro @abate commented on commit 406e7e43
      Owner

      For consistency on the UI, I think we can use the same time picker for dates only (I think) ... Or the flatpickr for periods (minor comment, not really important)

      Edited by pietro
  • pietro approved this merge request

    approved this merge request

  • pietro mentioned in commit 4871eb95

    mentioned in commit 4871eb95

  • merged

  • Please register or sign in to reply
    Loading