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

[UPLOAD-1147] remove open #1609

Merged
merged 8 commits into from
Jul 1, 2024
Merged

[UPLOAD-1147] remove open #1609

merged 8 commits into from
Jul 1, 2024

Conversation

krystophv
Copy link
Member

@krystophv krystophv commented Feb 12, 2024

for UPLOAD-1147 removes the open dependency in favor of Electron/Node native mechanisms for opening external URLs

@krystophv krystophv requested a review from gniezen February 12, 2024 22:40
gniezen
gniezen previously approved these changes Feb 13, 2024
Copy link
Member

@gniezen gniezen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! :shipit:

* master: (47 commits)
  v2.57.0-power-only-fix.1
  fix issue where not all devices are displaying the power-only USB warning
  v2.57.0
  v2.56.1-update-deps.6
  bump platform-client
  use final versions of sundial and platform-client
  v2.56.1-enforce-patient-limit.6
  notarize fixes
  v2.56.1-enforce-patient-limit.5
  [UPLOAD-1179] style update to handle extremely long clinic lists
  v2.56.1-enforce-patient-limit.4
  [UPLOAD-1187] trigger login success action after fetch clinics action
  v2.56.1-enforce-patient-limit.3
  [UPLOAD-1187] optional chain for ui prop to not potentially not exist
  v2.56.1-enforce-patient-limit.2
  [UPLOAD-1187] design feedback/port WEB fix
  Updated config.yml
  Updated config.yml
  satisfy the linter
  v2.56.1-enforce-patient-limit.1
  ...
@gniezen gniezen merged commit cdc8e5e into master Jul 1, 2024
1 of 3 checks passed
@gniezen gniezen deleted the UPLOAD-1147-remove-open branch July 1, 2024 11:19
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.

2 participants