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

Feat migrate from storyblok id to storyblok UUID #786

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

strawHat121
Copy link

Resolves #781

What changes did you make and why did you make them?

  • Removed all occurences of storyblokId and replaced it with storyblokUuid
  • Updated the unit test cases.

Did you run tests? Share screenshot of results:

Screenshot 2025-02-25 at 5 32 40 PM

How did you find us? (GitHub, Google search, social media, etc.):

Github

@strawHat121
Copy link
Author

Hi @kyleecodes

So I tried my best to make all the changes and I think I have covered them all here. I also updated the test cases wherever it was required.

Since this required making changes in test cases also, I am not sure about whatever I have done is correct or not. So if possible can you please take a look at my PR?

if it looks ok, then I will start making changes in the frontend repo and will run the full unit tests as well as e2e tests there using playwright.

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.

Migrate backend code to use storyblokUuid
1 participant