Skip to content

Conversation

0slb
Copy link
Contributor

@0slb 0slb commented Aug 15, 2025

Implementation that allows to set src in a <img> tag to the URL of an image.

@0slb
Copy link
Contributor Author

0slb commented Aug 15, 2025

@microsoft-github-policy-service agree

@ultmaster
Copy link
Contributor

Looks excellent. Would you add tests and update tsdoc for this feature?

@0slb
Copy link
Contributor Author

0slb commented Aug 15, 2025

Updated docs and added a test for URL image fetching.
I also updated the npm command shown in the python docs

@ultmaster
Copy link
Contributor

please pull from main and fix the tests.

function generatePythonFile(jsonData: ComponentSpec[]): string {
let pythonCode = `# This file is auto-generated from component documentation.
# Do not edit manually. Run \`npm run build-comment\` to regenerate.
# Do not edit manually. Run \`npm run docs:build\` to regenerate.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please revert unnecessary changes.

@ultmaster
Copy link
Contributor

The tests need to be fixed before merging.

@0slb
Copy link
Contributor Author

0slb commented Aug 18, 2025

fixed tests for image fetching in Webpage

@ultmaster ultmaster added this to the v0.1 milestone Aug 25, 2025
@ultmaster
Copy link
Contributor

I've just created an image url for the testing, please use this one:

https://raw.githubusercontent.com/ultmaster/poml-test-fixtures/e2bed155890e4cf853a515a990660328799ee5e3/image/gpt-5-random-image.png

Also merge from main again. Apologize for the prettier just merged.

@0slb
Copy link
Contributor Author

0slb commented Aug 26, 2025

updated the image URL for testing

@ultmaster ultmaster merged commit 3739961 into microsoft:main Aug 27, 2025
20 of 21 checks passed
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

Successfully merging this pull request may close these issues.

2 participants