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

Fix: Handle Craft Links with Custom Domains by Implementing getCraftSlug Function #35

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

Conversation

mehmetabak
Copy link

The Craft application used for displaying notes has introduced a feature allowing users to share links with custom domains. Previously, links followed the pattern "https://www.craft.do/s/" with a 23-character slug, which was sliced to extract the necessary ID. However, this approach fails with the new custom domain links, causing issues in preview images and other functionalities. This affects new users configuring the Craft settings for the project as well.

To address this problem, I have implemented a getCraftSlug function in the lib directory. This function correctly extracts the slug from both the old and new link formats, ensuring the application functions correctly without errors. This change ensures that both legacy and new links are handled seamlessly, maintaining the integrity of the application for all users.

Copy link

vercel bot commented Jul 11, 2024

@memoli0 is attempting to deploy a commit to the zuolan's projects Team on Vercel.

A member of the Team first needs to authorize it.

@mehmetabak
Copy link
Author

@izuolan , can you check it?

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.

1 participant