-
Notifications
You must be signed in to change notification settings - Fork 102
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: repo collection (final) #730
base: master
Are you sure you want to change the base?
Conversation
Hi @wxharry, the modal view you provided for testing purpose doen't work on my computer. |
Hi, can you provide more info from the console? I cannot reproduce this on my side. |
@wxharry Sorry, I might not reload Hypercrx from |
* feat: developed RepoCollection Button through antd component * feat: implement LocalStorage,SetDefault,DeleteCollection,AddCurRepo feature * style: yarn run prettier for index.scss * feat: refactor view * feat: implement storage feature * feat: implement collection modal from popup button * feat: implement collection modal using tabs and repo list beside * feat: implement collection modal from repository dropdown click * feat: solve reopen modal problem * feat: implement collection editor function (quick import need to be done) * feat: add judgment for whether there is initial data * feat: create CollectionEditor component * feat: use GitHub REST API to get repo and its description * feat: implement CollectionEditor add and edit feature * feat: solve formatting and naming issues * feat: refactor data structure * refactor: implement the collection button in the original GitHub style (step 1) * chore: update yarn.lock * fix: update module import * chore: update yarn.lock * fix: duplicate rendering after tab switches * fix: prevent duplicate finally * refactor: give CollectionModal a separate directory * refactor: CollectionList (30%) and AddToCollections (1%) * feat: support basic interactions with CollectionButton * chore: disable charts-design * chore: allow overflow when there are many exsiting collections * chore: add comments * chore: add manage button in AddToCollections * refactor: a possible final directory * chore: support to open modal from CollectionList * fix: fix typo * feat: add confirm check before collection deletion * feat: set default key for Table and add Radio for quick import (User/Organization) * refactor: move custom type definition to context.ts * refactor: use constate to better manage locally global state * feat: store and useStore for repo-collection * chore: update repo name after turbo:restore * feat: replace some part with real data * feat: AddToCollections(100%) * fix: a simple lock mechanism to prevent concurrent updates in store.ts * feat: implement CollectionDisplayModal and rename filename * refactor: rename view.tsx * feat: add divider for footer in AddToCollections * refactor: remove redundant initialization * refactor: rollback and delete displayModal * feat: implement of CollectionManageModal * feat: implement of CollectionEditor confirm * feat: use await/async function * chore: fix typo and add validator for duplicate collection name * style: maximize the collection modal size --------- Co-authored-by: Lam Tang <tangyenan@gmail.com>
* feat: Implement brief layouts and chart styles * feat: Implemented BarChart、StackedBarChart, PieChart * feat: Implemented BoxplotChart, SankeyChart * feat: Implemented GridLayout, Simple Chart Display, Implemented 'Highlight Specific Charts by Clicking the Menu Bar' * feat: Implemented NumericPanel and made adjustments to the BarChart,StackedBarChart and Boxplot * feat: Removed the deprecated gridlayout file * feat: Modify the 'get newest month' function * feat: merge collection Modal
Please remember to improve your comments in the code. Make sure it is descriptive and simple in English, ask gpt to do it would be a good choice. |
Resolve #756 |
Hi~ It is now possible to add a repository by directly entering the repository name. A new add option demo.mp4 |
After #713 and #716 are fully merged into this branch, the PR should be merged into master.