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

Favorite Page: Snapshots should be displayed when the language is changed #24221

Closed
Tracked by #22343
nollymar opened this issue Feb 27, 2023 · 4 comments · Fixed by #24234
Closed
Tracked by #22343

Favorite Page: Snapshots should be displayed when the language is changed #24221

nollymar opened this issue Feb 27, 2023 · 4 comments · Fixed by #24234
Assignees
@josemejias07
Labels
QA : Approved QA : Passed Internal Release : 23.03 Team : Scout Type : Defect
Milestone
2023.03

Comments

@nollymar
Copy link
Contributor

Parent Issue

#22343

Problem Statement

If the language is changed in Edit Mode, existing favorite pages are loaded with blank snapshots.

Steps to Reproduce

  1. Using full starter create some favorite pages in English
  2. Open an existing page in Edit Mode and change the language to Spanish
  3. Go back to the Favorite Pages view http://localhost:8080/dotAdmin/#/pages and see how the existing favorite pages are loaded. Snapshot images are broken.
Screen.Recording.2023-02-27.at.1.00.09.PM.mov

Acceptance Criteria

Snapshots in favorite pages view should be displayed correctly no matter the language selected in Edit Mode.

dotCMS Version

23.03

Proposed Objective

Please Select

Proposed Priority

Please Select

External Links... Slack Conversations, Support Tickets, Figma Designs, etc.

No response

Assumptions & Initiation Needs

No response

Quality Assurance Notes & Workarounds

No response

Sub-Tasks & Estimates

No response
@nollymar nollymar added this to the 2023.03 milestone Feb 27, 2023
@alfredo-dotcms
Copy link
Contributor

alfredo-dotcms commented Feb 27, 2023 •
edited
Loading

@nollymar after debuging no idea why changing the LANG in Edit mode breaks fetching the resource (the path remains the same, is like in some way changing the Lang makes the server lock files to a current specific language), right now I'm using the attr screenshot, but if I change it to screenshotVersion it works!
image

but no idea if that's ok

@alfredo-dotcms alfredo-dotcms self-assigned this Feb 27, 2023
@nollymar
Copy link
Contributor Author

add this param to the path: ?language_id=

alfredo-dotcms added a commit that referenced this issue Feb 28, 2023
@alfredo-dotcms
#24221 fix Snapshots should be displayed when the language is changed
a46a700
@alfredo-dotcms
Copy link
Contributor

PR: https://github.com/dotCMS/core/pull/24234/files

@alfredo-dotcms alfredo-dotcms linked a pull request Feb 28, 2023 that will close this issue
Issue 24221 screenshot not showing when language changed #24234
Merged
3 tasks
nollymar pushed a commit that referenced this issue Mar 2, 2023
@alfredo-dotcms
#24221 fix Snapshots should be displayed when the language is changed (…
5ce847f 
…#24234)
@nollymar nollymar mentioned this issue Mar 2, 2023
Closed
fmontes added a commit that referenced this issue Mar 6, 2023
@jcastro-dotcms @victoralfaro-dotcms
@rjvelazco @alfredo-dotcms @fmontes @manuelrojas @zulqarnainvd @hassan-mustafa-baig
Merging latest release-23.03 into master (#24264)
164d3e8 
* Update release version for dotcms-ui and dotcms-webcomponents

* Modify dotcmsReleaseVersion to 23.03, coreWebReleaseVersion, webComponentsReleaseVersion to rc and dot-cicd branch version to release-23.03

* Update branch in git submodule to release-23.03

* Excluding from triggering test at folder: dotCMS/src/main/webapp/html

* #23977 Block editor freeze scroll on tippy menus show (#24180)

* dev: freeze scroll on show forms #23977

* clean up

* refactor

* clean up v2

* feedback

* clean up

* #24172 Refresh Page portlet's data when site changed (#24175)

* #24031 fix block editor not saving (#24222)

* Fix #24032 Bring it to 22.03 (#24229)

* Fix #24032 Bring it to 22.03

* Build de lib

---------

Co-authored-by: Manuel Rojas <manuel.rojas.21@gmail.com>

* #24240 Fix data-access tests in 23.03 (#24244)

* #24240 fix test cases

* #24240 fix lint

* fire release docker image generation

* #24221 fix Snapshots should be displayed when the language is changed (#24234)

* Fix #23449 show dotAssets in folder listing

* Fixing merge conflict in

* Merging conflicts

* Merging conflicts

---------

Co-authored-by: victoralfaro-dotcms <victor.alfaro@dotcms.com>
Co-authored-by: Rafael Velazco <rjvelazco21@gmail.com>
Co-authored-by: alfredo-dotcms <37185433+alfredo-dotcms@users.noreply.github.com>
Co-authored-by: Freddy Montes <751424+fmontes@users.noreply.github.com>
Co-authored-by: Manuel Rojas <manuel.rojas.21@gmail.com>
Co-authored-by: zulqarnainvd <113915849+zulqarnainvd@users.noreply.github.com>
Co-authored-by: hassan-mustafa-baig <111717530+hassan-mustafa-baig@users.noreply.github.com>
fmontes added a commit that referenced this issue Mar 9, 2023
@jcastro-dotcms @victoralfaro-dotcms
@rjvelazco @alfredo-dotcms @fmontes @manuelrojas @zulqarnainvd @hassan-mustafa-baig @wezell
Merging latest release-23.03 into master - PR3 (#24322)
e3b2916 
* Update release version for dotcms-ui and dotcms-webcomponents

* Modify dotcmsReleaseVersion to 23.03, coreWebReleaseVersion, webComponentsReleaseVersion to rc and dot-cicd branch version to release-23.03

* Update branch in git submodule to release-23.03

* Excluding from triggering test at folder: dotCMS/src/main/webapp/html

* #23977 Block editor freeze scroll on tippy menus show (#24180)

* dev: freeze scroll on show forms #23977

* clean up

* refactor

* clean up v2

* feedback

* clean up

* #24172 Refresh Page portlet's data when site changed (#24175)

* #24031 fix block editor not saving (#24222)

* Fix #24032 Bring it to 22.03 (#24229)

* Fix #24032 Bring it to 22.03

* Build de lib

---------

Co-authored-by: Manuel Rojas <manuel.rojas.21@gmail.com>

* #24240 Fix data-access tests in 23.03 (#24244)

* #24240 fix test cases

* #24240 fix lint

* fire release docker image generation

* #24221 fix Snapshots should be displayed when the language is changed (#24234)

* Fix #23449 show dotAssets in folder listing

* Bock Editor: Allow user to insert videos from local files#23863 (#24248)

* dev: remove fileSize limit and fit the video in the block editor #23863

* clean up

* Fix 24032: Adding for Internal QA (#24255)

* #24032 Adding Fix for Internal QA

* #24032 Adding PR Feedback

* #24032 Adding PR Feedback

* #24032 Adding PR Feedback

* #24032 Logging error

* dotCMS/core#issue-20432 adding const

* dotCMS/core#issue-20432 adding const

* #24032 validation schema

* Adding empty extensions

* Adding empty extensions

* Get all modules no actions required

* Adding optional validation for actions

* Adding optional validation for actions

* Adding id insted of name

* Adding Type for promise

* Adding dot-cms-block-editor

* Adding lock file

---------

Co-authored-by: Freddy Montes <751424+fmontes@users.noreply.github.com>

* #22151 make the message better (#24041)

* #22151 make the message better

* #22151 hides token button if aws

* #24173 Create dialog for 'Create Page' button (#24215)

* #24173 Create dialog for 'Create Page' button

* #24173 Create dialog for 'Create Page' button - tests

* #24173 Create dialog for 'Create Page' button - feedback

* #24173 Create dialog for 'Create Page' button - more feedback

* #23901 fixes the git hash resolution (#23902)

* #23889 fixed content permission load issue (#24296)

* #24308 Can't search for contentlets in edit page palette (#24316)

* #24308 fixed contentlets search

* #24308 refactor test case

* #24308 refactor code

* Update package-lock

* Fix versioning

* Fix versioning and FE tests

* Build

* Add line

* Had to dedupe because some internal pck upgrade nx bin

---------

Co-authored-by: victoralfaro-dotcms <victor.alfaro@dotcms.com>
Co-authored-by: Rafael Velazco <rjvelazco21@gmail.com>
Co-authored-by: alfredo-dotcms <37185433+alfredo-dotcms@users.noreply.github.com>
Co-authored-by: Freddy Montes <751424+fmontes@users.noreply.github.com>
Co-authored-by: Manuel Rojas <manuel.rojas.21@gmail.com>
Co-authored-by: zulqarnainvd <113915849+zulqarnainvd@users.noreply.github.com>
Co-authored-by: hassan-mustafa-baig <111717530+hassan-mustafa-baig@users.noreply.github.com>
Co-authored-by: Will Ezell <will@dotcms.com>
@josemejias07
Copy link

Approved QA - Tested on 23.03_7d8de130_SNAPSHOT // Docker // macOS 13.0 // FF v110.0.1

nollymar pushed a commit that referenced this issue Mar 15, 2023
@alfredo-dotcms @victoralfaro-dotcms
@rjvelazco @fmontes @manuelrojas @zulqarnainvd @hassan-mustafa-baig @wezell
#23792 Pages portlet - Bugs found (#24265)
5868866 
* Update release version for dotcms-ui and dotcms-webcomponents

* Modify dotcmsReleaseVersion to 23.03, coreWebReleaseVersion, webComponentsReleaseVersion to rc and dot-cicd branch version to release-23.03

* Update branch in git submodule to release-23.03

* Excluding from triggering test at folder: dotCMS/src/main/webapp/html

* #23977 Block editor freeze scroll on tippy menus show (#24180)

* dev: freeze scroll on show forms #23977

* clean up

* refactor

* clean up v2

* feedback

* clean up

* #24172 Refresh Page portlet's data when site changed (#24175)

* #24031 fix block editor not saving (#24222)

* Fix #24032 Bring it to 22.03 (#24229)

* Fix #24032 Bring it to 22.03

* Build de lib

---------

Co-authored-by: Manuel Rojas <manuel.rojas.21@gmail.com>

* #24240 Fix data-access tests in 23.03 (#24244)

* #24240 fix test cases

* #24240 fix lint

* fire release docker image generation

* #24221 fix Snapshots should be displayed when the language is changed (#24234)

* Fix #23449 show dotAssets in folder listing

* Bugs fixed:
- When I search by URL I need to type exactly the full url to get a match
- If I edit a page using right click and save it, the new changes aren't always displayed in the table and the context menu doesn't list the options correctly

* Bock Editor: Allow user to insert videos from local files#23863 (#24248)

* dev: remove fileSize limit and fit the video in the block editor #23863

* clean up

* Fix 24032: Adding for Internal QA (#24255)

* #24032 Adding Fix for Internal QA

* #24032 Adding PR Feedback

* #24032 Adding PR Feedback

* #24032 Adding PR Feedback

* #24032 Logging error

* dotCMS/core#issue-20432 adding const

* dotCMS/core#issue-20432 adding const

* #24032 validation schema

* Adding empty extensions

* Adding empty extensions

* Get all modules no actions required

* Adding optional validation for actions

* Adding optional validation for actions

* Adding id insted of name

* Adding Type for promise

* Adding dot-cms-block-editor

* Adding lock file

---------

Co-authored-by: Freddy Montes <751424+fmontes@users.noreply.github.com>

* #22151 make the message better (#24041)

* #22151 make the message better

* #22151 hides token button if aws

* #24173 Create dialog for 'Create Page' button (#24215)

* #24173 Create dialog for 'Create Page' button

* #24173 Create dialog for 'Create Page' button - tests

* #24173 Create dialog for 'Create Page' button - feedback

* #24173 Create dialog for 'Create Page' button - more feedback

* #23792 feedback

* #23792 More Feedback

* #23792 More Feedback - unit tests

* #23792 More Feedback v2

---------

Co-authored-by: victoralfaro-dotcms <victor.alfaro@dotcms.com>
Co-authored-by: Rafael Velazco <rjvelazco21@gmail.com>
Co-authored-by: Freddy Montes <751424+fmontes@users.noreply.github.com>
Co-authored-by: Manuel Rojas <manuel.rojas.21@gmail.com>
Co-authored-by: zulqarnainvd <113915849+zulqarnainvd@users.noreply.github.com>
Co-authored-by: hassan-mustafa-baig <111717530+hassan-mustafa-baig@users.noreply.github.com>
Co-authored-by: Will Ezell <will@dotcms.com>
fmontes added a commit that referenced this issue Mar 28, 2023
@jcastro-dotcms @victoralfaro-dotcms
@rjvelazco @alfredo-dotcms @fmontes @manuelrojas @zulqarnainvd @hassan-mustafa-baig @wezell @erickgonzalez @jdotcms
Merging 23 03 to master (#24470)
da2f8b1 
* Update release version for dotcms-ui and dotcms-webcomponents

* Modify dotcmsReleaseVersion to 23.03, coreWebReleaseVersion, webComponentsReleaseVersion to rc and dot-cicd branch version to release-23.03

* Update branch in git submodule to release-23.03

* Excluding from triggering test at folder: dotCMS/src/main/webapp/html

* #23977 Block editor freeze scroll on tippy menus show (#24180)

* dev: freeze scroll on show forms #23977

* clean up

* refactor

* clean up v2

* feedback

* clean up

* #24172 Refresh Page portlet's data when site changed (#24175)

* #24031 fix block editor not saving (#24222)

* Fix #24032 Bring it to 22.03 (#24229)

* Fix #24032 Bring it to 22.03

* Build de lib

---------

Co-authored-by: Manuel Rojas <manuel.rojas.21@gmail.com>

* #24240 Fix data-access tests in 23.03 (#24244)

* #24240 fix test cases

* #24240 fix lint

* fire release docker image generation

* #24221 fix Snapshots should be displayed when the language is changed (#24234)

* Fix #23449 show dotAssets in folder listing

* Bock Editor: Allow user to insert videos from local files#23863 (#24248)

* dev: remove fileSize limit and fit the video in the block editor #23863

* clean up

* Fix 24032: Adding for Internal QA (#24255)

* #24032 Adding Fix for Internal QA

* #24032 Adding PR Feedback

* #24032 Adding PR Feedback

* #24032 Adding PR Feedback

* #24032 Logging error

* dotCMS/core#issue-20432 adding const

* dotCMS/core#issue-20432 adding const

* #24032 validation schema

* Adding empty extensions

* Adding empty extensions

* Get all modules no actions required

* Adding optional validation for actions

* Adding optional validation for actions

* Adding id insted of name

* Adding Type for promise

* Adding dot-cms-block-editor

* Adding lock file

---------

Co-authored-by: Freddy Montes <751424+fmontes@users.noreply.github.com>

* #22151 make the message better (#24041)

* #22151 make the message better

* #22151 hides token button if aws

* #24173 Create dialog for 'Create Page' button (#24215)

* #24173 Create dialog for 'Create Page' button

* #24173 Create dialog for 'Create Page' button - tests

* #24173 Create dialog for 'Create Page' button - feedback

* #24173 Create dialog for 'Create Page' button - more feedback

* #23901 fixes the git hash resolution (#23902)

* #23889 fixed content permission load issue (#24296)

* #24308 Can't search for contentlets in edit page palette (#24316)

* #24308 fixed contentlets search

* #24308 refactor test case

* #24308 refactor code

* #24243 fixed squash between hint icon  and label (#24298)

* dev: update block editor (#24323)

* [Issue-23863]: Fix #23863 Max Dimensions for Videos and Images (#24338)

* #23863 adding fix for max dimensions

* Adding editor

* Fix label

---------

Co-authored-by: Freddy Montes <751424+fmontes@users.noreply.github.com>

* mockPrintWriter (#24352)

* #24263 prevent empty save block editor field (#24342)

* Fix #24230 Block editor markdown content with javascript code is breaking the app (#24359)

* #24230

* #24230  clean up

* #24230 moving logic to the JSP

* [Issue 24370] block editor crashing with allowed blocks (#24375)

* #24370  Fixing undefined this

* #24370  Fixing undefined this

* Fix #23863 block editor allow user to insert videos from local files fix size (#24374)

* dev: prevent close form while uploading asset #23863

* fix: large videos preview

* clean up

* update block editor

* feedback

* dev: update dotcms-block-editor.js file

* Block Editor: Allow users to insert videos from external sources#23861 (#24409)

* dev: validate video url

* dev: add video validation #23861

* clean up

* fix merge conflicts

* update dotcms-block-editor.js

* dev: add support link to unknown errors

* Update core-web/libs/block-editor/src/lib/extensions/asset-form/components/dot-external-asset/utils/index.ts

* Update core-web/libs/block-editor/src/lib/extensions/asset-form/components/dot-external-asset/utils/index.ts

* update dotcms-block-editor.js

---------

Co-authored-by: Freddy Montes <751424+fmontes@users.noreply.github.com>

* Fix #24361 block editor code block should not show suggestions or placeholder (#24403)

* #24361 Blocking suggestions and placeholder

* #24361 Blocking suggestions and placeholder

* #24361 Blocking suggestions and placeholder

* #24361 Blocking suggestions and placeholder

* build file

---------

Co-authored-by: Freddy Montes <751424+fmontes@users.noreply.github.com>

* #24422 fixes render

* Update LTS check

* Fix #23863 let user cancel upload (#24417)

* dev: let user cancel upload #23863

* update dotcms-block-editor.js

* clean up

* feedback

* update dotcms-block-editor.js

* dev: upload automatically after selecting the video

* Fix #24263: get the correct input element for block editor (#24429)

* Fix #23449 Filter does not work on the image selector for image fields 

* Moved code to a new branch

* Fix for the DB query

* Fixed failing ITs also addressed the PR feedback

* Added javadoc

* Fixed the failing test

* Fixed some ITs

* Fix postman tests

* Fixed postman tests

* Addressed PR feedback

* Making sonar happy

* Adding more code changes.

* Removed unnecessary file

* Fixed luceneQuery to fix failing ITs

* Adding `showOnMenu` term to lucene query.

* Updating reindex policy in Integration Test to wait for the ES reindex to be done when saving the test files.

* Adding temporary logging to the Integration Test

* Force indexation.

* Print more info.

* Removing `SYSTEM_HOST` from DB query.

* Fixing Integration Test.

---------

Co-authored-by: Jose Castro <jose.castro@dotcms.com>

* Fix #24299 now the get dependencies do not throw an exception when the story block value is not a json (#24439)

* Bock Editor: Allow user to insert videos from local files #23863 (#24451)

* dev: support video file and dotAsset #23863

* update dotcms-block-editor.js file

* dev: update label to Library and dotcms-block-editor.js file

* Block editor value [Object, object] fix (#24474)

* Updating branch references

* dev: rename video node (#24477)

* dev: restore data mock time (#24478)

* Fix conflict

* Fix lock

* Point to master

* Fix lock

* Fixing merging conflict.

---------

Co-authored-by: victoralfaro-dotcms <victor.alfaro@dotcms.com>
Co-authored-by: Rafael Velazco <rjvelazco21@gmail.com>
Co-authored-by: alfredo-dotcms <37185433+alfredo-dotcms@users.noreply.github.com>
Co-authored-by: Freddy Montes <751424+fmontes@users.noreply.github.com>
Co-authored-by: Manuel Rojas <manuel.rojas.21@gmail.com>
Co-authored-by: zulqarnainvd <113915849+zulqarnainvd@users.noreply.github.com>
Co-authored-by: hassan-mustafa-baig <111717530+hassan-mustafa-baig@users.noreply.github.com>
Co-authored-by: Will Ezell <will@dotcms.com>
Co-authored-by: erickgonzalez <erick.gonzalez@dotcms.com>
Co-authored-by: Jonathan <jonathan.sanchez@dotcms.com>
@wezell wezell closed this as completed May 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@josemejias07 josemejias07

Labels
QA : Approved QA : Passed Internal Release : 23.03 Team : Scout Type : Defect
Projects
dotCMS - Product Planning
Archived in project
Milestone
2023.03
Development

Successfully merging a pull request may close this issue.

Issue 24221 screenshot not showing when language changed dotCMS/core
6 participants
@jgambarios @wezell @nollymar @alfredo-dotcms @josemejias07 @damen-dotcms