Skip to content

[WIP] additional front-end features [onto development]#8

Closed
gabriellesc wants to merge 0 commit intodevelopmentfrom
ui-dev
Closed

[WIP] additional front-end features [onto development]#8
gabriellesc wants to merge 0 commit intodevelopmentfrom
ui-dev

Conversation

@gabriellesc
Copy link
Collaborator

@gabriellesc gabriellesc commented Aug 25, 2017

Port of CP PR #34: uoft-tapp/cp#34

- [x] checkbox range selection

  • table in-column searching
  • email templating with mangled applicant contract URL
    doesn't seem to be possible, since fetching the mangled contract URL would expose a vulnerability
    @michellemtchai what do you think?

- [ ] session selection
- [x] modifying sessional pay

@michellemtchai:

@gabriellesc I can open up an admin only route GET /decision/:utorid/:position_id like before. This way the students gets their mangled routes and we get the readable routes for the admin.

@gabriellesc:

@michellemtchai what would that route return?

@michellemtchai:

@gabriellesc It would render the student-facing view. It's the same page the applicant sees when they click on the mangled link.

@gabriellesc:

@michellemtchai This would be for the purpose of inserting the mangled link into an email to send to people, not just for seeing the student-facing view

@michellemtchai:

@gabriellesc That can be an extra route or it can be part of the /offers fetch. Which do you prefer?

@gabriellesc
Copy link
Collaborator Author

@michellemtchai I suppose it makes sense to make it part of the /offers fetch?

@gabriellesc gabriellesc changed the base branch from master to development August 26, 2017 04:12
@gabriellesc gabriellesc changed the title [WIP] additional front-end features [WIP] additional front-end features [onto development] Aug 27, 2017
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