Skip to content

feat: add stac-browser v5.x support - #281

Open
jjfrench wants to merge 2 commits into
mainfrom
feat/support-stac-browser-v5.x
Open

feat: add stac-browser v5.x support#281
jjfrench wants to merge 2 commits into
mainfrom
feat/support-stac-browser-v5.x

Conversation

@jjfrench

@jjfrench jjfrench commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

⚠️ Checklist if your PR is changing anything else than documentation

  • Posted the link to a successful manually triggered deployment workflow (successful including the resources destruction)

Merge request description

  • Modifies the stac-browser buildapp to support v5.0.0 of the radiantearth stac-browser by checking the tag version and passing props into their respective builds v3/v4,v5

How you can test it

By commenting out the bucket section of the construct, you can easily target the build portion of the stac-browser construct

// test-build.ts
import * as cdk from 'aws-cdk-lib';

// lib/stac-browser/index.ts
import { StacBrowser } from '../lib/stac-browser/index.ts';


function main() {

    // Throwaway app/stack purely to satisfy the Construct tree — never synthesized
    // to a template file or deployed.
    const app = new cdk.App();
    const stack = new cdk.Stack(app, 'TestStack');

    // Instantiating this runs buildApp() as a side effect of the constructor.
    new StacBrowser(stack, 'TestStacBrowser', {
        stacCatalogUrl: "https://stac.maap-project.org",
        githubRepoTag: "v3.2.0", // v5.0.0
        websiteIndexDocument: 'index.html',
    });

    console.log(`   Try: npx serve ./stac-browser/dist`);
}

main();

you might need a tsconfig.json, e.g.

{
  "compilerOptions": {
    "target": "ES2022",
    "module": "NodeNext",
    "moduleResolution": "NodeNext",
    "allowImportingTsExtensions": true,
    "strict": false,
    "skipLibCheck": true
  },
  "include": ["./**/*.ts"]
}

npx ts-node test-build.ts

npx serve ./stac-browser/dist

@jjfrench
jjfrench marked this pull request as ready for review August 4, 2026 17:08
@jjfrench jjfrench self-assigned this Aug 4, 2026
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