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

fix: load all do apps paginated #2863

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

Conversation

lauhon
Copy link

@lauhon lauhon commented Dec 10, 2024

Description 📣

The backend that delivers all apps for a given DO api key does only fetch the fist 20 projects because of the pagination that the DO api offers

Type ✨

  • Bug fix
  • New feature
  • Improvement
  • Breaking change
  • Documentation

Tests 🛠️

On my arch I was unable to execute the .dev dockerfile. So this has not been tested yet

   OK: Importing 'infisical/infisical-cli' repository RSA key...
 NOPE: Checking if upstream install config is OK ...
#0 4.307  ^^^^: ... Failed to fetch configuration for your OS distribution release/version.
#0 4.308  >>>>:
#0 4.310  >>>>: It looks like we don't currently support your distribution release and
#0 4.310  >>>>: version. This is something that we can fix by adding it to our list of
#0 4.310  >>>>: supported versions (see contact us below), or you can manually override
#0 4.310  >>>>: the values below to an equivalent distribution that we do support:
#0 4.310  >>>>:
#0 4.311  >>>>: ... distro=alpine  version=3.21.0  codename=<empty>  arch=aarch64  
#0 4.311  >>>>:
#0 4.312  >>>>: You can force this script to use a particular value by specifying distro,
#0 4.312  >>>>: version, or codename via environment variable. E.g., to specify a distro
#0 4.312  >>>>: such as Alpine (v3.8), use the following:
#0 4.312  >>>>:
#0 4.312  >>>>:   <curl command> | distro=alpine version=3.8.0 codename= sudo -E bash
#0 4.312  >>>>:
#0 4.312  !!!!: Oh no, your setup failed! :-( ... But we might be able to help. :-)
#0 4.312  !!!!: 
#0 4.312  !!!!: You can contact Infisical for further assistance.
#0 4.312  !!!!: 
#0 4.312  !!!!: URL: https://github.com/Infisical/infisical
#0 4.312  !!!!: 
#0 4.312  !!!!: Last 20 log lines from .csm_setup_XXXHcjpAg (might not be errors, nor even relevant):
#0 4.312  !!!!:
#0 4.316 
------
failed to solve: process "/bin/sh -c apk add --no-cache bash curl && curl -1sLf   'https://dl.cloudsmith.io/public/infisical/infisical-cli/setup.alpine.sh' | bash   && apk add infisical=0.8.1 && apk add --no-cache git" did not complete successfully: exit code: 1

Notes

The frontend might want to support a type search in the dropdown. Scrolling and looking through >200 apps is not fun :D

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