Conversation
|
@decidim/core tests fail due to something about CodeClimate, can you check it please? I'm not sure what's going on... |
9db666d to
d244e82
Compare
d244e82 to
26795bf
Compare
leio10
left a comment
There was a problem hiding this comment.
Great work, very easy to review! ❤️
| </div> | ||
|
|
||
| <%= render "decidim/proposals/admin/proposal_notes/proposal_notes" %> | ||
| <% if allowed_to? :create, :proposal_note %> |
There was a problem hiding this comment.
Shouldn't admins who can't create proposal notes be allowed to view existing notes?
There was a problem hiding this comment.
I kept the current behavior, and currently if you can't create proposal notes you can't view the current ones...
| rubyzip (>= 1.2.2) | ||
| seven_zip_ruby (1.2.5) | ||
| simplecov (0.18.0) | ||
| simplecov (0.17.1) |
There was a problem hiding this comment.
Jeje, I thought this would happen when I saw the first commit 😄
There was a problem hiding this comment.
Yeah, I had a little problem with dependencies and rebasing from master 😞
There was a problem hiding this comment.
Oh, I had problems with CodeClimate coverage report when I updated to simplecov 0.18, I thought that was the problem: qltysh-archive/test-reporter#418
There was a problem hiding this comment.
Yes, we had to lock simplecov to v0.17.1 so that CI jobs could end successfully, see #5673 😄
tramuntanal
left a comment
There was a problem hiding this comment.
Hi @mrcasals good job, just take a look to my comments.
Also, only if you have the time, it will be great to have an edit icon at the top-right corner 😄
decidim-proposals/app/helpers/decidim/proposals/admin/proposals_helper.rb
Show resolved
Hide resolved
decidim-proposals/app/views/decidim/proposals/admin/proposals/_proposal-tr.html.erb
Show resolved
Hide resolved
decidim-proposals/app/views/decidim/proposals/admin/proposals/show.html.erb
Show resolved
Hide resolved
f51b366 to
aee4ede
Compare
|
@tramuntanal I've added the link to the edit page (see screenshot in the PR description), and I've updated the screenshot relating a meeting with the PR, so that it shows they appear. Regarding the icon, as said in #5671 (comment), I've moved the discussion to #5630, but please let's not block this PR for that, I'll open a new one if needed 🙏 Can you re-review this please? |
* Add needed scopes to comments * Update dependencies * Add proposal show page * Add notes to proposal show page * Change proposal links * Improve styles * Move proposal answers to show page * Show votes and endorsements rankings * Fix locales * Fix tests to use the new page * Add tests for proposal info * Add tests for ranking helper * Remove unused test * Remove unused file * Show proposal related meetings * Show message button next to user author * Show related attachments * Remove unused keys * Add changelog * Don't contact author if it's the current user * Undo Gemfile.lock changes * Crop endorsers list to 5 * Add link to edit proposal
🎩 What? Why?
This PR adds a proposal show page for the admin section. this page lists some proposal data, and merges the proposal notes and answer pages into it. This means we can get rid of the notes and answer pages, since they have been merged into the show page.
📌 Related Issues
📋 Subtasks
CHANGELOGentry📷 Screenshots (optional)
If the current admin can edit the proposal, a link to edit it is shown next to the proposal title:
