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

Available tag #433

Merged
merged 3 commits into from
Oct 27, 2023
Merged

Available tag #433

merged 3 commits into from
Oct 27, 2023

Conversation

Janaka-Steph
Copy link
Contributor

@Janaka-Steph Janaka-Steph commented Oct 26, 2023

It closes #413

@what-the-diff
Copy link

what-the-diff bot commented Oct 26, 2023

PR Summary

  • Introduction of App Import Type
    A new type of import has been added called App. This assists in bringing in other dependencies or information required by our system.

  • Creation of 'tags' State
    There's a new state named tags, which will be used to manage and keep track of various information tags within the system.

  • The Shift from 'apps' to 'tags'
    Several parts of the code have been updated to use the newly created tags state instead of the earlier app directly, making the system more efficient and flexible. This includes requests, newSearch and onSelectChange variables and the operational logic within two effects (useEffect). Additionally, this change is also reflected in the returned map functions and selectedApps variable.

  • Enhancement in Styling Logic
    The styling logic of certain div elements has been improved. The class bg-[#F4A597] rounded in the className prop of the div elements now uses search.get(app.id) operation instead of apps.get(app.id). This makes the applied styles more linked to the actual retrieved application data.



@Janaka-Steph Janaka-Steph marked this pull request as draft October 26, 2023 17:10
@Janaka-Steph Janaka-Steph marked this pull request as ready for review October 27, 2023 06:14
@tiero
Copy link
Contributor

tiero commented Oct 27, 2023

/Users/tiero/projects/prem/prem-app/src/modules/service/components/Service.tsx
  3:10  warning  'useParams' is defined but never used  @typescript-eslint/no-unused-vars

✖ 1 problem (0 errors, 1 warning)

@tiero tiero merged commit 4469919 into prem-research:main Oct 27, 2023
1 check passed
@Janaka-Steph Janaka-Steph deleted the tag branch October 30, 2023 09:45
tiero pushed a commit that referenced this pull request Nov 1, 2023
* Available tag

* filter available + rm appId

* rm unused var
@tiero tiero mentioned this pull request Nov 3, 2023
tiero added a commit that referenced this pull request Nov 6, 2023
* use v1 branch of prem registry

* Fix chat history (#439)

* Available tag (#433)

* Available tag

* filter available + rm appId

* rm unused var

* gha: workflow_dispatch with manual version and branch name

* gha: debug

* gha: use version as number

* v0.1.3

* remove mp3 (#432)

* Update to TanStack Query v5 (#443)

* fix: binary_url precedence logic (#450)

* fix: binary_url precedence logic

* fix: lint

* Remove isFetching (#460)

* fix: match premd routes with slash in docker controller (#455)

* fix: gracefully handle process kill (#442)

* GHA: split Tauri and Docker in two workflows (#462)

* chore: add docker-compose to test app, daemon and gateway

* docker-compose.gateway point to stable premd

* v0.1.4

* fix: buggy state fetch (#471)

* v0.1.5

---------

Co-authored-by: Janaka-Steph <kakobaba1212@protonmail.com>
Co-authored-by: Biswaroop <biswaroop08@gmail.com>
Co-authored-by: Swarnim Arun <swarnimarun11@gmail.com>
Janaka-Steph added a commit to Janaka-Steph/prem-app that referenced this pull request Nov 7, 2023
* use v1 branch of prem registry

* Fix chat history (prem-research#439)

* Available tag (prem-research#433)

* Available tag

* filter available + rm appId

* rm unused var

* gha: workflow_dispatch with manual version and branch name

* gha: debug

* gha: use version as number

* v0.1.3

* remove mp3 (prem-research#432)

* Update to TanStack Query v5 (prem-research#443)

* fix: binary_url precedence logic (prem-research#450)

* fix: binary_url precedence logic

* fix: lint

* Remove isFetching (prem-research#460)

* fix: match premd routes with slash in docker controller (prem-research#455)

* fix: gracefully handle process kill (prem-research#442)

* GHA: split Tauri and Docker in two workflows (prem-research#462)

* chore: add docker-compose to test app, daemon and gateway

* docker-compose.gateway point to stable premd

* v0.1.4

* fix: buggy state fetch (prem-research#471)

* v0.1.5

---------

Co-authored-by: Janaka-Steph <kakobaba1212@protonmail.com>
Co-authored-by: Biswaroop <biswaroop08@gmail.com>
Co-authored-by: Swarnim Arun <swarnimarun11@gmail.com>
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.

Add tag option "Show only available models" to filter not available one
2 participants