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(specs): add support for widgets / banners in search for the csharp client #3870

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

Conversation

Kalmar99
Copy link

@Kalmar99 Kalmar99 commented Oct 2, 2024

fix #3869

🧭 What and Why

The csharp client currently does not support Widgets / Banners as part of the RenderingContent model, however this feature is now available in the dashboard / merchandising studio. This PR adds the missing models required for parsing banners from the api response.

Changes included:

Adds the following models:

  • Banner
  • BannerImage
  • BannerImageUrl
  • BannerLink
  • Widgets
    Also added the missing widgets field to the RenderingContent model

@Kalmar99 Kalmar99 requested a review from a team as a code owner October 2, 2024 12:33
@millotp
Copy link
Collaborator

millotp commented Oct 3, 2024

Hey, all the files you edited are generated files, all the files in this folder are generated, can you edit the spec in specs/search please ? You don't to push the generated code, the CI will do it for you

@Kalmar99
Copy link
Author

Kalmar99 commented Oct 3, 2024

Ah, i dident notice that this was all auto generated 😅 Will update my PR in a bit

@Kalmar99
Copy link
Author

Kalmar99 commented Oct 3, 2024

I updated the PR now, removing the models and updating the schema, however i am struggling with setting up the environment for building this. Followed your guide as outlined here https://api-clients-automation.netlify.app/docs/setup-repository but running into a very ambiguous error with the linter

Error: command failed: yarn specs:lint common at run (C:\projects\other\api-clients-automation\scripts\common.ts:114:13) at async lintCommon (C:\projects\other\api-clients-automation\scripts\specs\format.ts:28:3) at async buildSpecs (C:\projects\other\api-clients-automation\scripts\specs\index.ts:149:3) at async Command.<anonymous> (C:\projects\other\api-clients-automation\scripts\cli\index.ts:125:5)

Any chance you could look over the schema and try to build it to see if i did it correctly? 😇

@millotp millotp changed the title feat(csharp) add support for widgets / banners in search for the csharp client feat(specs): add support for widgets / banners in search for the csharp client Oct 3, 2024
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]: RenderingContent is missing widgets section
2 participants