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

[NEW] Gitlab Component Kit #98

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

Conversation

samad-yar-khan
Copy link
Contributor

@samad-yar-khan samad-yar-khan commented Mar 25, 2022

Issues

Mentioned in the issue #72

Proposed changes (including videos or screenshots)

  • New Reusable Gitlab Components Kit

  • Data fetching method in cms (can be re-used when switching from strapi) to provide a better experience for community builders. All data fetching for the gitalb kit for a repository will be done by a a single gitlabKit() function.

  • Introduce Single <Gitlab ... > tag which can be used to fetch different components of the kit depending on props/parameters. <Gitlab type={'members'} gitlabData={props.gitlabData} />

  • Introduce single gitlabKitData(project_id ,[..needs]) function which can fetch relevant data based on the components we wish to use on a particular page.

  • New Gitlab Merge Request Component

    merge-example

  • New Gitlab Project Members Component

members-example

  • New Gitlab Project Overview Component

overview

  • New Gitlab Project Issues Component

issues-example

2022-03-27.16-56-32.mp4
  • New Gitlab Page to showcase the components kit which can be accessed at /gitlab
2022-03-27.14-51-24.mp4

@samad-yar-khan samad-yar-khan changed the title [WIP] [NEW] Gitlab Component Kit [NEW] Gitlab Component Kit Mar 27, 2022
@lgtm-com
Copy link

lgtm-com bot commented Mar 27, 2022

This pull request introduces 1 alert when merging acf335c into 0930180 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@samad-yar-khan
Copy link
Contributor Author

@Sing-Li I have added all the necessary to the Gitlab Components Kit , added documentation and have also created a page to showcase the component. Let me if know if there are any changes 😅

@lgtm-com
Copy link

lgtm-com bot commented Apr 3, 2022

This pull request introduces 1 alert when merging 03e351d into 6541e17 - view on LGTM.com

new alerts:

  • 1 for Useless assignment to local variable

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.

1 participant