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.

Sign up for GitHub

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

Jump to bottom

Issues editing Post in closed topic #1352

Closed
JN-Jones opened this issue Aug 31, 2014 · 1 comment · Fixed by #1439
Closed

Issues editing Post in closed topic #1352

JN-Jones opened this issue Aug 31, 2014 · 1 comment · Fixed by #1439
Assignees
@Sama34
Labels
b:1.8 Branch: 1.8.x p:medium Priority: Medium. Issues to be resolved with normal preference s:resolved Status: Resolved. Solution implemented or scheduled t:bug Type: Bug. An issue causing error / flaw / malfunction
Milestone
1.8.1

Comments

@JN-Jones
Copy link
Contributor

Since I don't have admin permissions to reply and post on closed threads, the edit button on my posts should probably disappear.  

When I use quick edit, rather than blocking my edit I see what is in the attached image.  

Hitting save then puts my old message back in and puts an error message popup on the top right, which isn't horrid. 

This is pretty low level since nothing bad really happens.  

Original thread: Issues editing Post in closed topic
@JN-Jones JN-Jones added bug and removed confirmed labels Aug 31, 2014
@JN-Jones JN-Jones added this to the 1.8.1 milestone Aug 31, 2014
@Sama34 Sama34 self-assigned this Sep 15, 2014
@Sama34 Sama34 mentioned this issue Sep 15, 2014
Merged
@Sama34 Sama34 added s:resolved Status: Resolved. Solution implemented or scheduled and removed s:confirmed Status: Confirmed. Retested and found the issue exists labels Sep 15, 2014
DiogoParrinha pushed a commit that referenced this issue Sep 27, 2014
Merge pull request #1439 from Sama34/fix-1352
fec01e4 
Fix #1352 Issues editing Post in closed topic
@DiogoParrinha
Copy link
Contributor

Merged PR. For reference, @Sama34 fixed the same issue with the delete button.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Sama34 Sama34

Labels
b:1.8 Branch: 1.8.x p:medium Priority: Medium. Issues to be resolved with normal preference s:resolved Status: Resolved. Solution implemented or scheduled t:bug Type: Bug. An issue causing error / flaw / malfunction
Projects
None yet
Milestone
1.8.1
Development

Successfully merging a pull request may close this issue.

Fix #1352 Issues editing Post in closed topic
3 participants
@DiogoParrinha @Sama34 @JN-Jones