department-of-veterans-affairs / va.gov-team

Public resources for building on and in support of VA.gov. Visit complete Knowledge Hub:
https://depo-platform-documentation.scrollhelp.site/index.html
281 stars 196 forks source link

[spike] Investigate using GitHub codespaces #87670

Open Mottie opened 3 weeks ago

Mottie commented 3 weeks ago

Value Statement

As a person interested in reviewing app changes I want to evaluate the app before it gets to staging So that I don't have to rush to review, test and iterate changes before these changes get to production


Background Context

While updating the Higher-Level Review content, we made changes that we'd like to review before it makes it to staging, then out to production. Production deployment is now at noon, so we now feel more rushed in the morning.

Adding a feature flag for the changes turned out to be more complicated than we anticipated. One page adds hint text to a radio, and since that only allows a string value, we have no easy method to update the uiSchema to add or remove the hint based on a feature toggle. To work around this issue, we'd need to change from using the schema to a custom page, which is more work than is needed for a simple hint text change.

Evaluation of codespaces may allow us to review and iterate on the changes from a page outside of the staging flow. This would be similar to review instances, but hopefully less flaky.

Acceptance Criteria

Tasks

Definition of Ready

Definition of Done

eileen-coforma commented 2 weeks ago

Difficult to get API server running. CORS problems. Other dev using Codespaces suggested using mock API, but might not working. Timeboxed for another couple hours.

Might have to default to using review instances.

Mottie commented 2 weeks ago

I think I've exhausted my investigation on codespaces:

Steps to get running:

Pros:

Cons:

eileen-coforma commented 2 weeks ago

Robin to write some documentation before closing.

eileen-coforma commented 2 weeks ago

Not using Codespaces because it does not work for our purposes.

Mottie commented 20 hours ago

Completed preliminary work on testing new features on VA.gov (feature toggles vs review instance vs codespaces vs duplicating the app) and shared with the other vfs-engineers for feedback