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(front): allow filtering from ui by project #218

Merged
merged 1 commit into from
May 13, 2020

Conversation

ekelen
Copy link
Collaborator

@ekelen ekelen commented May 13, 2020

TODO:

  • bug: handle no driver or artifact kind is in global state filters
  • decide if having an 'all' option for apps and artifact kinds is necessary (decided no for this MR)
  • more manual tests, might finally need to implement error boundary (Handle React component failures gracefully #72 )
  • allow filtering by project OR artifact kinds from filter modal
  • add gomobile-ipfs-demo (is there a shorter name for this?) to UI
  • filter by project gomobile-ipfs-demo triggers query ?artifact_kinds=2&artifact_kinds=1&build_driver=3
  • filter by project chat triggers query ?artifact_kinds=1&build_driver=1

Style tweaks:

  • add style for disabled filters in modal

Blocks #202
Closes #201

@trafico-bot trafico-bot bot added the 🚧 WIP Still work-in-progress, please don't review and don't merge label May 13, 2020
@ekelen ekelen self-assigned this May 13, 2020
@ekelen ekelen force-pushed the feat/web-add-default-project-filters branch 2 times, most recently from 3920d98 to 9004059 Compare May 13, 2020 08:04
@ekelen ekelen requested a review from moul May 13, 2020 08:05
@ekelen ekelen changed the title WIP: feat(front): allow filtering from ui by project feat(front): allow filtering from ui by project May 13, 2020
@trafico-bot trafico-bot bot added 🔍 Ready for Review Pull Request is not reviewed yet and removed 🚧 WIP Still work-in-progress, please don't review and don't merge labels May 13, 2020
get project filter from driver and artifact kind

remove branch from state and hardcode

cleanup branch display in modal

add toggler for filtering by project

add style for disabled filters in modal

enable add and remove of projects in filter ui

setup binding between driver, artifact kinds, and project

build_drivers->build_driver to match api

add chat build driver to default url params

remove debug text

apply filters from ui

remove duplicates from filters

organize imports in filter modal

parse url query for drivers

add default query if user unselects all filters

reset build drivers when not searching by project

add APK to gomobile project filter defaults
@ekelen ekelen force-pushed the feat/web-add-default-project-filters branch from 9004059 to cf01800 Compare May 13, 2020 08:09
@trafico-bot trafico-bot bot added ✅ Approved Pull Request has been approved and can be merged and removed 🔍 Ready for Review Pull Request is not reviewed yet labels May 13, 2020
@moul moul merged commit da333e4 into master May 13, 2020
@moul moul deleted the feat/web-add-default-project-filters branch May 13, 2020 09:33
@trafico-bot trafico-bot bot added ✨ Merged Pull Request has been merged successfully and removed ✨ Merged Pull Request has been merged successfully ✅ Approved Pull Request has been approved and can be merged labels May 13, 2020
@github-actions
Copy link

🎉 This PR is included in version 2.42.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

Add other applications to UI filters
2 participants