Skip to content

Conversation

@ethancrawford
Copy link
Collaborator

I'm not totally familiar with the process of importing the latest icons into Tabler core/preview, so @codecalm / @BG-Software-BG or other folks more familiar with the process, feel free to comment on anything that's not right - but I decided to test whether I could manually update https://github.com/tabler/tabler/blob/dev/shared/data/icons.json by running preview/.build/import-icons.mjs on the command line.

At first, I encountered an error because the path here did not exist:

join(__dirname, `../shared/data/icons-info.json`),

(which in my case was returning):
Error: ENOENT: no such file or directory, open '/home/ethan/repos/tabler/preview/shared/data/icons-info.json'
instead of generating the correct directory, /home/ethan/repos/tabler/shared/data/icons-info.json.

Once I updated the path on lines 29 and 38 of /preview/.build/import-icons.mjs, the /shared/data/icons.json and /shared/data/icons-info.json files appeared to update successfully.

I was intending to test whether updating /shared/data/icons.json would fix #2297 - however I'm also not sure how to view a local dev version of the documentation app - is this currently possible?

@changeset-bot
Copy link

changeset-bot bot commented Apr 30, 2025

⚠️ No Changeset found

Latest commit: db0a17e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link

vercel bot commented Apr 30, 2025

@ethancrawford is attempting to deploy a commit to the Tabler Team on Vercel.

A member of the Team first needs to authorize it.

@ethancrawford ethancrawford requested a review from codecalm April 30, 2025 06:54
@ethancrawford
Copy link
Collaborator Author

😂 😂
That's what I get for not checking the existing open PR list. Oops. Sorry!

@ethancrawford ethancrawford deleted the update-icons-import branch April 30, 2025 07:03
@ethancrawford ethancrawford removed the request for review from codecalm April 30, 2025 07:06
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.

[BUG] Switch icon examples are broken

1 participant