Skip to content

Credit Roles #644

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Credit Roles #644

wants to merge 1 commit into from

Conversation

jyhein
Copy link
Contributor

@jyhein jyhein commented Jun 13, 2025

  • Adds Credit Roles and degrees of contribution to contributor form and landing page
  • Adds 2 db tables
  • Downloads translations

{{ t('submission.submit.creditRoles.description', {}) }}
</div>
<div class="pkpFormField__control pkpFormField--creditRoles__control">
<PkpTable aria-label="Credit Roles">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one needs be localised. I see you followed the FieldAffiliation, where its also not localised.. will address that.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you follow the improvements I just did to the FieldAffiliation to follow same pattern? I think thats best fit here.

ff9c645#diff-a7100dddb36405c9a76f5469cce92e8186a9122a3c4e14568aa63f60fed85aaa

@@ -0,0 +1,133 @@
<template>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please add it also to the storybook?

Have we had designs for it? Just wondering if there is reference for me to look at.

Copy link
Contributor

@bozana bozana Jun 25, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How is it added to the storybook?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checkout FieldAffiliations.stories.js for example

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok. How do I test stories? I haven't done this before

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jardakotesovec , I added the storybook

@jyhein jyhein force-pushed the f857_credit branch 2 times, most recently from 92deb00 to 6003915 Compare July 2, 2025 10:30
Copy link
Contributor

@jardakotesovec jardakotesovec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jyhein Thank you for the updates, looking great. Sorry for delay, just returned from the vacation.

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.

3 participants