Skip to content
This repository has been archived by the owner on Sep 26, 2024. It is now read-only.

Feature: leverage the API for filters #380

Closed
1 of 2 tasks
bdougie opened this issue Oct 19, 2022 · 4 comments · Fixed by #377
Closed
1 of 2 tasks

Feature: leverage the API for filters #380

bdougie opened this issue Oct 19, 2022 · 4 comments · Fixed by #377

Comments

@bdougie
Copy link
Member

bdougie commented Oct 19, 2022

Type of feature

🍕 Feature

Current behavior

This was implemented in the alpha branch prior but got deleted in a merge conflict fix. It will need to be reimplemented.

a471317

Suggested solution

Copy and paste the changes as is:

  1. Create the src/hooks/useRepositoriesList.ts
  2. Include new types src/vite-types.d.ts (if needed)
  3. use useRepositoriesList in RepoWrap
  4. Set "Recent" as the default filter instead of "popular"

a471317

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs
@bdougie
Copy link
Member Author

bdougie commented Oct 19, 2022

Here is a twitch clip of the filter working

https://clips.twitch.tv/GentleSuccessfulGalagoMau5-CD_abMjk45zxxlXk

This was referenced Oct 19, 2022
@bdougie
Copy link
Member Author

bdougie commented Oct 19, 2022

With the merge of this PR #381, we can focus this issue to make recent the default and in this order (pictured).

Screen Shot 2022-10-19 at 4 23 11 PM

This work should be done on beta and not alpha after this is merged #378

github-actions bot pushed a commit that referenced this issue Oct 20, 2022
## [2.26.0-beta.7](v2.26.0-beta.6...v2.26.0-beta.7) (2022-10-20)

### 🍕 Features

* make recent the default filter ([#377](#377)) ([3ace891](3ace891)), closes [#376](#376) [#380](#380)
@github-actions
Copy link

🎉 This issue has been resolved in version 2.26.0-beta.7 🎉

The release is available on:

Your semantic-release bot 📦🚀

github-actions bot pushed a commit that referenced this issue Oct 28, 2022
## [2.26.0](v2.25.0...v2.26.0) (2022-10-28)

### 🐛 Bug Fixes

* add menu drop to generate token on console ([#367](#367)) ([dce7249](dce7249)), closes [#364](#364)
* add open a bug link when repo loading fails ([#349](#349)) ([50efbc8](50efbc8)), closes [#307](#307)
* alpha build ([#360](#360)) ([06a5ef2](06a5ef2))
* correct [#321](#321) changes ([0080521](0080521))
* correct alpha mistakes ([#381](#381)) ([f917ac2](f917ac2))
* correct latest alpha ([6e82aa6](6e82aa6))
* correct merge conflicts from bad heads ([566f753](566f753))
* correct upstream fast forward ([2897584](2897584))
* remove swr local storage cache provider ([#371](#371)) ([334081c](334081c)), closes [#365](#365)
* rename everything from Post to Repo  ([#356](#356)) ([0267009](0267009)), closes [#313](#313)
* trigger build on alpha ([#359](#359)) ([dbbbea5](dbbbea5))

### 🍕 Features

* add loading skeleton for the repoWrap component ([#374](#374)) ([058beab](058beab)), closes [#357](#357)
* add myVotes filter for repo list ([#384](#384)) ([e0e70ed](e0e70ed)), closes [#334](#334)
* added custom tailwind variables and removed anti patterns ([#337](#337)) ([549e676](549e676)), closes [#315](#315)
* adds localstorageprovider ([#361](#361)) ([f030796](f030796)), closes [#335](#335)
* auth/session API call temporary at Navbar component. ([#379](#379)) ([841b635](841b635)), closes [#366](#366)
* display repos' contributors avatars ([#383](#383)) ([ecebb1e](ecebb1e)), closes [#333](#333)
* **api:** leverage the api over sdk ([#321](#321)) ([a471317](a471317)), closes [#312](#312) [#312](#312) [#312](#312)
* make recent the default filter ([#377](#377)) ([3ace891](3ace891)), closes [#376](#376) [#380](#380)
* new favicon ([#389](#389)) ([67ab18b](67ab18b))
* New Favicon Replacement ([#363](#363)) ([bb680a6](bb680a6)), closes [#362](#362)
* Update SVG favicon ([#369](#369)) ([bf78ca0](bf78ca0)), closes [#368](#368)
@github-actions
Copy link

🎉 This issue has been resolved in version 2.26.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants