owid / etl

A compute graph for loading and transforming OWID's data
https://docs.owid.io/projects/etl
MIT License
81 stars 22 forks source link

⚠️ explorers: markdown → html #1643

Closed lucasrodes closed 1 year ago

lucasrodes commented 1 year ago

In #1422, we only looked at descriptions using HTML in ETL. However, there are still some charts and explorers using HTML: Those from manual configs in explorers, datasets that were manually imported in the past, and datasets imported via importers.

This issue tries to address this for Explorers. I've included the list of explorers below; could you check these as those are fixed?

Guidelines

There are different types of explorers, and reviewing these might be different.

List of explorers to review

(only public ones)

lucasrodes commented 1 year ago

some work for explorers based on templating done here: : https://github.com/owid/owid-content/pull/31

lucasrodes commented 1 year ago

@paarriagadap Hey Pablo, I've already implemented some changes in https://github.com/owid/owid-content/pull/31. Some of it affects your work; it would be helpful if you could briefly review it. Thanks!

pabloarosado commented 1 year ago

@lucasrodes, after merging https://github.com/owid/owid-content/pull/33 all explorers in the list should be cleared of HTML tags. Shall we close the issue?

lucasrodes commented 1 year ago

Thanks for your work, @pabloarosado @paarriagadap

Closing this!