Migrate backend code to use storyblokUuid

[kyleecodes]

Overview

The backend currently uses both storyblokId and storyblokUuid. We need to migrate all code to use storyblokUuid to maintain consistency.

❗️ Blockers: While there are no blockers for this issue, this issue should ideally be completed alongside this frontend issue.

The database migration is handled in this separate issue.

Action Items

• Audit: identify all instances where storyblokId is used in the backend code.
• Update code: remove storyblokId from entities, interfaces, and all code, including affected services and controllers. Some functions may need refactored to reflect reduction in number of arguments required.
• Test: report any errors in issue or PR discussions.
• Modify unit tests calling storyblokId, then run unit tests.
• Test functionality in the UI (check that user paths work, functions related to the modifications work, and check the backend / frontend logs for errors)
• You may run integration tests, however, maintainers will run integration tests with more coverage from the frontend on your open PR, so be available for discussion there.

Resources

• What is the difference between the id, uuid, and _uid?

[strawHat121]

Please assign this to me as well.

[github-actions]

Thank you @strawHat121 you have been assigned this issue!
Please follow the directions in our Contributing Guide. We look forward to reviewing your pull request. ✨

---

Support Chayn's mission? ⭐ Please star this repo to help us find more contributors like you!
Learn more about our impact and sign-up for our volunteer programsto join our mission!. 🌸

[strawHat121]

Hi @kyleecodes

I was going through the codebase and had some questions on this issue.

1. From what I can see, the storyBlokId is used in resource.service.ts, session.service.ts and a few more places. So I just need to find all occurences and replace it with storyblokUuid?
2. How to test out these things? I tried to trigger the current functions by running both frontend and backend locally and completing a course, but still I was not able to trigger the function.

[kyleecodes]

@strawHat121 great questions! I've updated the issue description with more details and upgraded this issue from moderate to advanced complexity level. If still interested in this issue, I suggest taking on the database migration issue linked in the description (this can be done separately, but it helps for testing). Otherwise, you can follow the updated testing steps in the description. Ultimately, this issue may require frequent communication with Chayn devs to notify us of errors and edge cases. We will keep track of this issue and respond promptly. If errors arise with replacing all instances, we can still merge a PR that replaces as much as possible, and create new issues for more complex replacements.