Closed JakeWags closed 7 months ago
Name | Link |
---|---|
Latest commit | 5637779b4afe9bd21e05f0fee6f5fe219431e82d |
Latest deploy log | https://app.netlify.com/sites/upset2/deploys/65e8a4e5b771fe0008066b5b |
Deploy Preview | https://deploy-preview-303--upset2.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Does this PR close any open issues?
Closes #301 Depends #302
Give a longer description of what this PR addresses and why it's needed
Previously, there was a strange discrepancy between the downloaded json data setnames and the alt-text generated setnames. This was due to the fact that the alt-text config generation function was not adjusting the element name field prior to export.
By adding this to the alt text, the output seems to be as expected.
Tested with Simpsons and Movie dataset.
Provide pictures/videos of the behavior before and after these changes (optional)
@elizaan Please confirm that this is correct:
Are there any additional TODOs before this PR is ready to go?
TODOs:
~&~