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(web): Open huggingface.co link on settings page in new tab #16470

Merged
merged 2 commits into from
Mar 1, 2025

Conversation

lusu007
Copy link
Contributor

@lusu007 lusu007 commented Mar 1, 2025

Description

On the admin settings page, within the machine learning section under "Smart Search," there is a link that redirects users to Immich's page on Hugging Face. Previously, when this link was clicked, it would open in the same tab as the admin settings page, which was inconvenient. Users often lost all unsaved changes while trying to view documentation. To address this issue, the link has now been updated to open in a new tab.

How Has This Been Tested?

  • Click hugginface.co link on admin settings page

Screenshots (if appropriate)

Not appropriate

Checklist:

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if applicable
  • I have no unrelated changes in the PR.
  • I have confirmed that any new dependencies are strictly necessary.
  • I have written tests for new code (if applicable)
  • I have followed naming conventions/patterns in the surrounding code
  • All code in src/services uses repositories implementations for database calls, filesystem operations, etc.
  • All code in src/repositories/ is pretty basic/simple and does not have any immich specific logic (that belongs in src/services)

@alextran1502 alextran1502 enabled auto-merge (squash) March 1, 2025 22:51
@alextran1502 alextran1502 merged commit 6cc1978 into immich-app:main Mar 1, 2025
41 of 42 checks passed
@lusu007 lusu007 deleted the fix/open-in-new-tab branch March 2, 2025 18:00
knechtandreas added a commit to knechtandreas/immich that referenced this pull request Mar 2, 2025
…n-grid

* main: (22 commits)
  feat: add album keyboard shortcuts (immich-app#16442)
  fix(web): Default to context search on web (immich-app#16485)
  fix(mobile): Updated formatting of server address in networking (immich-app#16483)
  chore(mobile): rename log enum to lowercase (immich-app#16476)
  fix(web): Open huggingface.co link on settings page in new tab (immich-app#16470)
  fix(web): fix typos (immich-app#16466)
  fix(web): Fixed people list overflowing in advanced search (immich-app#16457)
  fix(web): unable to download live photo as anonymous user (immich-app#16455)
  fix(mobile): fix typos (immich-app#16456)
  chore: add 'not duplicate' checkbox to issue template (immich-app#16462)
  chore(mobile): post release task (immich-app#16437)
  feat(server): Shortened asset ID in storage template (immich-app#16433)
  chore: version v1.128.0
  fix: memories off by one (immich-app#16434)
  fix(server): stringify error log parameter to ensure correct overload (immich-app#16422)
  fix: duplicate memories (immich-app#16432)
  chore(server): trash e2e cleanup (immich-app#16423)
  fix(server): include deleted assets if searching offline assets (immich-app#16417)
  fix: user delete sync query sort by id (immich-app#16420)
  fix(mobile): background backup failing due to store (immich-app#16418)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants