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.

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

Issue when modifying dashboard including text tile #40

Closed
sampitcher opened this issue Jul 24, 2019 · 4 comments
Closed

Issue when modifying dashboard including text tile #40

sampitcher opened this issue Jul 24, 2019 · 4 comments

Comments

@sampitcher
Copy link

sampitcher commented Jul 24, 2019

Gazer version = v0.2.14
Looker version = 6.16.10

We have a customer set up on 2 instances, dev and prod. We are modifying content in dev instance, then promoting content to prod when changes have been made using gzr. We need to keep the dashboard slugs from dev to prod the same so links do not break..
The majority of dashboards move over fine, however if a dashboard contains a text tile we get this error message:

Modifying existing dashboard 2 E-commerce | Direct control tower in space 9
Error updating dashboard_element(27,{
  "body_text": "<p><a href=\"https://analytics.google.com/analytics/web/#/report/visitors-geo/a7823531w161529192p162626565/geo-segmentExplorer.segmentId=analytics.city&geo-table.plotKeys=%5B%5D/\" target=\"_blank\"><img src=\"https://s3-us-west-2.amazonaws.com/s.cdpn.io/1615306/Google%20Analytics%20logo_lockup_analytics_icon_vertical_black_2x.png\" alt=\"\" height=\"85\" /></a></p>",
  "dashboard_id": 2,
  "look_id": null,
  "merge_result_id": null,
  "note_display": null,
  "note_state": null,
  "note_text": null,
  "query_id": null,
  "refresh_interval": null,
  "result_maker": null,
  "subtitle_text": "",
  "title": "GA Icon",
  "title_hidden": false,
  "title_text": "",
  "type": "text"
})
Validation Failed
{:field=>"type", :code=>"invalid", :message=>"Result Maker IDs should not be provided for text elements", :documentation_url=>"http://docs.looker.com/"}```

So far I have to remove the text tile in dev content and then add it in manually in prod.

Thank you!
@HPNoha
Copy link

HPNoha commented Aug 13, 2019

Same here, on version 6.14
@sampitcher I noticed this doesn't occur on text tiles with body, so my workaround was to add an empty html body e.g <center></center>

@ghost
Copy link

ghost commented Oct 25, 2019

Also seeing this on 6.20.15.
We see this on text tiles with body, but no title or subtitle. Adding an empty title would make the tile too large for the dashboard.

@ghost
Copy link

ghost commented Oct 25, 2019

Although it only fails when it is attempting to overwrite an existing dashboard (using --force). If I move the existing dashboard to trash and delete from trash, the import works without error.

@drstrangelooker
Copy link
Collaborator

I just pushed gazer version 0.2.19. I believe that will fix this issue. If it does not fix we can reopen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants