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

Added image data URL handling to the excel cell content handler (#1237) #1239

Merged

Conversation

speckyspooky
Copy link
Contributor

The enhancement make the option of data url accessible for excel output which is located into text element based on image tag.

@speckyspooky speckyspooky added the Enhancement Small change to improve the current supported functionality label Mar 20, 2023
@speckyspooky speckyspooky added this to the 4.14 milestone Mar 20, 2023
@speckyspooky
Copy link
Contributor Author

Can I start the merge of the pull request?

@hvbtup
Copy link
Contributor

hvbtup commented Mar 24, 2023

Can I start the merge of the pull request?

Yes.

@wimjongman
Copy link
Contributor

Hey Thomas, since you are now a commiter, you can squash and merge the PR. Please use Squash and merge whenever merging so we do not get all the developer commits in.

The workflow is to first ask for a review. Then if you have a positive review, you can SQ&M.

@speckyspooky speckyspooky merged commit b3158c9 into eclipse-birt:master Mar 24, 2023
@speckyspooky
Copy link
Contributor Author

Thanks Henning & Wim! The merge is done with SQ&M.

@wimjongman
Copy link
Contributor

Congratulations on your first merge. I hope many more will follow. 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Small change to improve the current supported functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants