Skip to content

Add basic proof-of-concept app#4

Merged
matt-dray merged 1 commit intomainfrom
1-poc-trend
Sep 29, 2025
Merged

Add basic proof-of-concept app#4
matt-dray merged 1 commit intomainfrom
1-poc-trend

Conversation

@matt-dray
Copy link
Contributor

@matt-dray matt-dray commented Sep 26, 2025

Close #1.

  • Added very simple {bslib} app with a sidebar, where the user can select a scheme and mitigator.
  • Added a version of the rates trend plot, which updates as the user makes selections.
  • Included a warning for users so they know the app is in development.

This works locally, and I'd encourage you to try and run it (i.e. by running the contents of app.R).
Otherwise I can screenshare.

However, the deployed app fails to fetch data, which I think is again related to {azkit} and authorisation. I've set the environmental variables AZ_CONTAINER_INPUTS and AZ_STORAGE_EP, which I think is all I should need. Don't worry about it for now, @francisbarton, but let's take a look next week.

@matt-dray matt-dray added this to the v0.1.0 milestone Sep 26, 2025
@matt-dray matt-dray self-assigned this Sep 26, 2025
@matt-dray matt-dray added enhancement New feature or request priority: must MoSCoW priority labels Sep 26, 2025
@matt-dray matt-dray marked this pull request as ready for review September 26, 2025 20:55
@matt-dray matt-dray requested a review from DCEW as a code owner September 26, 2025 20:55
Copy link
Contributor

@DCEW DCEW left a comment

Choose a reason for hiding this comment

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

Works fine for me locally. Approving noting that deployment not yet fixed.

@matt-dray matt-dray merged commit c94a854 into main Sep 29, 2025
2 checks passed
@matt-dray matt-dray deleted the 1-poc-trend branch September 29, 2025 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request priority: must MoSCoW priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create proof-of-concept app

2 participants