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(cc-oauth-consumer): init component #1340

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

HeleneAmouzou
Copy link
Contributor

@HeleneAmouzou HeleneAmouzou commented Feb 18, 2025

Fix #1294

What does this PR do?

  • Introduce cc-oauth-consumer and cc-oauth-consumer-form components
    • The cc-oauth-consumer component display oAuth Consumer information
    • The cc-oauth-consumer-form is a creating / updating form (not ready yet)
  • Refactor of change-case helper to add snakeCase and camelCase

How to review?

  • cc-oauth-consumer
    • Check the code and the commits ( ⚠️ exept the wip commit with cc-oauth-consumer-form )
    • Check the stories in the preview (to do add preview link)
    • Play with demo-smart
    • Check if the design is close enough to the wireframe made by Marion
  • cc-oauth-consumer-form
    • TO DO

Things to be discussed

  • Should the component be renamed cc-oauth-consumer-info ?

TO DO

  • Fix typechecking in cc-oauth-consumer smart
  • Fix rights section loading mode
  • Add smart documentation
  • cc-oauth-consumer-form (wip)

@HeleneAmouzou HeleneAmouzou self-assigned this Feb 18, 2025
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.

cc-oauth-consumer: init component
1 participant