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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

admin can duplicate/copy meetings #3051

Merged
merged 4 commits into from
Mar 21, 2018
Merged

admin can duplicate/copy meetings #3051

merged 4 commits into from
Mar 21, 2018

Conversation

isaacmg410
Copy link
Contributor

@isaacmg410 isaacmg410 commented Mar 20, 2018

馃帺 What? Why?

This PR allows admins to duplicate or copy face-to-face meetings.

馃搶 Related Issues

馃搵 Subtasks

  • Add CHANGELOG entry
  • Add icon duplicate on index meetings
  • Add Command CopyMeeting
  • Add MeetingCopyForm
  • Add MeetingCopiesController
  • Add Views Admin MeetingCopies
  • Add new routes
  • Add specs

@ghost ghost assigned isaacmg410 Mar 20, 2018
@ghost ghost added the status: WIP label Mar 20, 2018
@codecov
Copy link

codecov bot commented Mar 20, 2018

Codecov Report

Merging #3051 into master will increase coverage by <.01%.
The diff coverage is 98.86%.

@@            Coverage Diff             @@
##           master    #3051      +/-   ##
==========================================
+ Coverage   98.62%   98.62%   +<.01%     
==========================================
  Files        1699     1704       +5     
  Lines       40602    40778     +176     
==========================================
+ Hits        40042    40216     +174     
- Misses        560      562       +2

@isaacmg410
Copy link
Contributor Author

@decidim/lot-core can you review it?

@mrcasals
Copy link
Contributor

@isaacmg410 no need to modify the routes? 馃槙

@isaacmg410
Copy link
Contributor Author

@mrcasals What do you mean? I have had to add new routes. Now I have put it in the descricption of issue, I forgot

@mrcasals
Copy link
Contributor

@isaacmg410 for some reason I missed the route addition, sorry about that 馃槥

Merging this! 馃槃 馃帀

@mrcasals mrcasals merged commit f1550a2 into master Mar 21, 2018
@mrcasals mrcasals deleted the 3044_duplicate_meetings branch March 21, 2018 08:39
rbngzlv added a commit that referenced this pull request Mar 21, 2018
* master:
  [RFC] Use cells for meeting m cards (#3022)
  Do not force Postgresql user to be admin when enabling trigram extension (#3053)
  Make organization reference_prefix required (#3056)
  admin can duplicate/copy meetings (#3051)
  Fix question form errors not being displayed (#3046)
  Erb whitespace cutting (#3047)
  Show debates statistics on space show and homepage (#3016)
  Fix broken translated field after form errors (#3026)
  Move decidim executable to "exe" folder (#3028)
  Friendlier buttons (#3027)
  Feedback needed after Endorsing when user has no user_groups (#2998)
  Fix seeding error on generator specs (#3021)
  fix spelling error in threshold (#3019)
  Migration plus seeds (#2933)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants