[PAGES-320] Clipboard component cannot be pasted to a page of different template than the one of the origin page Created: 29/Jul/20 Updated: 13/May/22 Resolved: 30/Sep/20 |
|
| Status: | Closed |
| Project: | Magnolia pages module |
| Component/s: | None |
| Affects Version/s: | 5.6.4, 5.7.1, 5.7.2, 6.2.2, 6.2.3 |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Neutral |
| Reporter: | Šimon Demočko | Assignee: | Unassigned |
| Resolution: | Not an issue | Votes: | 1 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | 1h 5m | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||||||||||||||
| Issue Links: |
|
||||||||||||||||||||
| Template: |
|
||||||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||||||||||||||
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
||||||||||||||||||||
| Date of First Response: | |||||||||||||||||||||
| Description |
Steps to reproduce
Expected resultsCan paste the copied text and image component Actual resultsPaste action is not available Development notes
Definition of done
|
| Comments |
| Comment by Chris Jennings [ 14/Sep/20 ] |
|
I believe this has been fixed in 6.2.3 by |
| Comment by Šimon Demočko [ 14/Sep/20 ] |
|
This ticket is about sth else. I filed it as a followup for |
| Comment by Chris Jennings [ 14/Sep/20 ] |
|
I'm afraid that's not what I'm seeing from a vanilla 6.2.3 with two light module page templates. A component created in page a, template type 1 CAN be pasted into page b, template type 2. Your steps for the travel demo do not work online because demo.magnol.... is still running 6.2.2 currently so this is a local trial. |
| Comment by Šimon Demočko [ 15/Sep/20 ] |
|
I did not try it on demo, but on a local instance on which I developed it. Nevertheless, I have a local instance running w/ 6.2.4-SNAPSHOTs now, tried the reprosteps and can still reproduce. Could you try it with the reprosteps on this page please? If it works for you with these, could you then send me your light modules config w/ reprosteps? Thanks |
| Comment by Chris Jennings [ 28/Sep/20 ] |
|
Hi. Sorry for the delay but I have added a light module (test.zip) with two page templates: template-a and template-b. website.type-a.yaml is a content export with the two pages pre-built. |
| Comment by Šimon Demočko [ 29/Sep/20 ] |
|
Hi, I tried your sample data, thanks a lot for those! You are right, it works with them. I double-checked the repro steps we filed which still reproduce it and indeed, the Text and Image used on the two templates were coming from different sources (one is mtk T&I the other one is travel T&I) and were not considered the same, which is why it wouldn't paste. Precisely what we mentioned in development notes to avoid :facepalm:. I'll double-check with Sang since it was his QA that pointed out this problem. We'll keep the ticket anyway to update the tests as mentioned in the acceptance criteria. Thanks again! |
| Comment by Šimon Demočko [ 30/Sep/20 ] |
|
Closing as not an issue, it was not reproducible with the right steps. The original steps were incorrect because they were trying to paste a component to a template that did not accept that component (confusion arose because there was a different component allowed there with the same name). Modifying test is split to a new ticket |