Skip to content

/series/{id}: admin should be able to add a comment #785

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

Closed
php-coder opened this issue Dec 30, 2017 · 2 comments
Closed

/series/{id}: admin should be able to add a comment #785

php-coder opened this issue Dec 30, 2017 · 2 comments
Assignees
Labels
impact/changelog This change should be reflected in the NEWS.txt file kind/feature priority/important
Milestone

Comments

@php-coder
Copy link
Owner

No description provided.

@php-coder php-coder added this to the next milestone Dec 30, 2017
@php-coder php-coder modified the milestones: next, 0.4.4 Mar 17, 2020
@php-coder php-coder self-assigned this Mar 17, 2020
@php-coder php-coder changed the title Series import: admin should be able to add a comment /series/{id}: admin should be able to add a comment Mar 17, 2020
@php-coder
Copy link
Owner Author

Instead of allowing admin to provide a comment during series import, let's make it generic and allow admin to add a comment to a series.

We also need to think about whether we allow update comment or not.

@php-coder php-coder added the impact/changelog This change should be reflected in the NEWS.txt file label Apr 12, 2020
@php-coder
Copy link
Owner Author

We also need to think about whether we allow update comment or not.

We can start from showing a form only when there is no comment.

php-coder added a commit that referenced this issue May 6, 2020
php-coder added a commit that referenced this issue May 24, 2020
Correction for 108acf7 commit.

Relate to #785
php-coder added a commit that referenced this issue Jun 13, 2020
Also fix the elements with duplicated ids.

Corrections to the following commits:
- 487ef61 (#1341, #1339)
- 86959e0 (#1342, #1340)
- 660e616 (#1338, #785)
- fb1df42 (#1344, #1343)
php-coder added a commit that referenced this issue Jun 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact/changelog This change should be reflected in the NEWS.txt file kind/feature priority/important
Projects
None yet
Development

No branches or pull requests

1 participant