-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Show Explore Extensions for self-hosted with static JSON fallback; remove legacy APPS_MARKETPLACE_API_URL #6391
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
Changes from all commits
014dd45
58d4b6d
fb7ea23
659295e
5bdf6dd
8420542
e09dabb
40b4680
77bedf6
c496b00
d80a55b
baa0045
36e3ed4
725c50a
65dc32d
fbc403d
7c0e3f7
ebd992c
063fa3b
aec79b7
3a07011
5aedd48
8740a7f
bf4374f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| --- | ||
| "saleor-dashboard": patch | ||
| --- | ||
|
|
||
| When App Store is not configured (env variable not set), Dashboard will not longer crash. Instead it will load | ||
| local catalog of apps and plugins to render them as a fallback | ||
lkostrowski marked this conversation as resolved.
Show resolved
Hide resolved
lkostrowski marked this conversation as resolved.
Show resolved
Hide resolved
lkostrowski marked this conversation as resolved.
Show resolved
Hide resolved
lkostrowski marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -108,9 +108,6 @@ runs: | |
| - name: Run tests | ||
| shell: bash | ||
| env: | ||
| ## backward compatibility for older versions | ||
| API_URI: ${{ inputs.API_URL }} | ||
|
Comment on lines
-111
to
-112
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. some 100 years old legacy, cleaned that up |
||
|
|
||
| API_URL: ${{ inputs.API_URL }} | ||
| BASE_URL: ${{ inputs.BASE_URL }} | ||
| E2E_USER_NAME: ${{ inputs.E2E_USER_NAME }} | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -600,7 +600,7 @@ const allActions: TriggerDescriptor[] = [ | |
| { | ||
| section: allMessages.extensionsSection, | ||
| name: allMessages.gotoExploreExtensions, | ||
| Component: ({ onClick }) => ( | ||
| Component: ({ onClick }: { onClick?: React.MouseEventHandler<HTMLAnchorElement> }) => ( | ||
| <Box onClick={onClick}> | ||
| <ActionLinkItem href={ExtensionsUrls.resolveExploreExtensionsUrl()}> | ||
| <FormattedMessage {...allMessages.gotoExploreExtensions} /> | ||
|
Comment on lines
600
to
606
|
||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.