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

Node's unpublished state is not represented in the UI #345

Closed
DevNils opened this Issue Mar 15, 2018 · 3 comments

Comments

Projects
None yet
2 participants
@DevNils

DevNils commented Mar 15, 2018

Gentics Mesh Version, operating system, or hardware.

  • v0.17.1

Operating System

  • Linux (Ubuntu 16.04)

JVM

  • Open JDK 1.8.0_151

Problem

When you unpublish a published node with a delete request via /{project}/nodes/{nodeUuid}/published, the unpublished state ist not represented in the UI of the CMS. You will see it in the API payload but not in the CMS ui.

Reproducer

  • Publish a node
  • Unpublish it with the rest API
  • Query the node with API and check the JSON payload
  • Check the node state in the CMS UI

Expected behaviour and actual behaviour

You will see that it shows "PUBLISHED" in the UI although the node has an unpublished state in the JSON payload.

@Jotschi Jotschi added UI bug labels Mar 15, 2018

@Jotschi

This comment has been minimized.

Member

Jotschi commented Mar 15, 2018

@DevNils Thanks for the report. This is a bug in the etag generation. I was able to reproduce it. The browser will use the node response from the browser cache instead of re-loading the node from mesh. Should be fairly easy to fix. I'll hopefully be able to include it in the next release.

@Jotschi Jotschi self-assigned this Mar 15, 2018

Jotschi added a commit that referenced this issue Mar 16, 2018

@Jotschi

This comment has been minimized.

Member

Jotschi commented Mar 16, 2018

@DevNils Fix is merged and will be included in the next release.

@Jotschi

This comment has been minimized.

Member

Jotschi commented Apr 6, 2018

The fix has been released.

@Jotschi Jotschi closed this Apr 6, 2018

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