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

Adds SVG to Json for SharePoint List Web Part #5338

Open
wants to merge 98 commits into
base: main
Choose a base branch
from

Conversation

LuiseFreese
Copy link
Contributor

  • New sample
  • Bug fix/update
  • Related issues: n/a

What's in this Pull Request?

Adds new sample that can Read SVGs from a specified library on a specified Site (property Pane), allows users to preview it and converts it into a format a SharePoint List's column will accept. Works with either convert & copy to clipboard (for further code manipulation) or can be directly applied to a Site, List, column of the users liking.

Node Version

18.20.4

Node version used:

Checklist

  • My pull request affects only ONE sample.
  • My sample builds without any warnings
  • I have updated the README.md file's Version history. For new samples, created a new README.md file matching this template
  • My README.md has at least one static high-resolution screenshot (i.e. not a GIF) located in the assets folder.
  • My README.md contains complete setup instructions, including pre-requisites and permissions required
  • My solution includes a .nvmrc file indicating the version of Node.js

Submitter Guidance (DELETE AFTER READING)

LuiseFreese and others added 30 commits October 16, 2024 10:15
@hugoabernier
Copy link
Collaborator

Hello @LuiseFreese , it was nice to see your demo and it's a delight to process this PR!

Your PR seems to include another web part (Bluesky?). Can you remove the second web part from the PR and submit that one as a separate PR?

If you need help to do so, I can process the PR and remove the second sample, and we'll process a separate PR for the Bluesky web part when you're ready.

image

image

Let me know how you wish to proceed and we'll make it happen!

@LuiseFreese
Copy link
Contributor Author

Hey @hugoabernier I removed the bluesky sample from this PR. Thank you and let me know if this sample needs further improvement.

@hugoabernier
Copy link
Collaborator

Hey @LuiseFreese, it looks like you re-added the Bluesky sample to the PR. Can you move it to a separate branch so I can process the sample?

@hugoabernier hugoabernier self-requested a review November 26, 2024 16:24
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.

3 participants