Skip to content

Update vaccine section - #884

Merged
benjystanton merged 23 commits into
mainfrom
update-vaccines-section
Sep 25, 2026
Merged

benjystanton merged 23 commits into
mainfrom
update-vaccines-section

Conversation

@benjystanton

@benjystanton benjystanton commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Updates

  • Fix drop-down where you can select a site on the Vaccines page. Only shown if you have 5 or more sites.
  • Make sure only or last site can't be deactivated. So don't show 'Deactivate site' link if it's the only site.
  • If batches are deactivated as a result of the site being deactivated, show the deactivation reason as 'Site deactivated'
  • On the Vaccines page, only show the Active sites and/or Deactivated sites tabs if there is at least 1 Active/Deactivated site
  • On the vaccine product page, only show the Active and/or Deactivated tabs if there is at least 1 Active/Deactivated batch
  • If no active batches are available and user clicks on vaccine product page, go straight to deactivated tab
  • Updated deactivate confirmation msg
  • Added error page for vax cannot be saved because site deactivated mid-flow
  • Added error page for generic problem deactivating a site
  • Added error page for problem deactivating a site because 2 users tried to do this at the same time
  • Updated General errors section of prototype index page to show the new errors

@frankieroberto
frankieroberto temporarily deployed to record-a-vac-update-vac-tv58vi September 24, 2026 08:32 Inactive
@benjystanton benjystanton changed the title Update vaccine sections Update vaccine section Sep 24, 2026
@frankieroberto
frankieroberto temporarily deployed to record-a-vac-update-vac-tv58vi September 24, 2026 08:40 Inactive
@frankieroberto
frankieroberto temporarily deployed to record-a-vac-update-vac-tv58vi September 24, 2026 08:41 Inactive
@frankieroberto
frankieroberto temporarily deployed to record-a-vac-update-vac-tv58vi September 24, 2026 08:46 Inactive
@frankieroberto
frankieroberto temporarily deployed to record-a-vac-update-vac-tv58vi September 24, 2026 08:52 Inactive
@frankieroberto
frankieroberto temporarily deployed to record-a-vac-update-vac-tv58vi September 24, 2026 09:02 Inactive
@frankieroberto
frankieroberto temporarily deployed to record-a-vac-update-vac-tv58vi September 24, 2026 09:15 Inactive
@frankieroberto
frankieroberto temporarily deployed to record-a-vac-update-vac-tv58vi September 24, 2026 09:39 Inactive
@frankieroberto
frankieroberto temporarily deployed to record-a-vac-update-vac-tv58vi September 24, 2026 09:46 Inactive
@frankieroberto
frankieroberto temporarily deployed to record-a-vac-update-vac-tv58vi September 24, 2026 10:04 Inactive
@frankieroberto
frankieroberto temporarily deployed to record-a-vac-update-vac-tv58vi September 24, 2026 10:43 Inactive
Deleting this error page as no longer needed.
@frankieroberto
frankieroberto temporarily deployed to record-a-vac-update-vac-tv58vi September 24, 2026 16:28 Inactive
Updated error message to clarify deactivation issues.
@frankieroberto
frankieroberto temporarily deployed to record-a-vac-update-vac-tv58vi September 24, 2026 16:30 Inactive
@frankieroberto
frankieroberto temporarily deployed to record-a-vac-update-vac-tv58vi September 24, 2026 16:33 Inactive
@frankieroberto
frankieroberto temporarily deployed to record-a-vac-update-vac-tv58vi September 24, 2026 19:13 Inactive
@frankieroberto
frankieroberto temporarily deployed to record-a-vac-update-vac-tv58vi September 25, 2026 07:53 Inactive
@frankieroberto
frankieroberto temporarily deployed to record-a-vac-update-vac-tv58vi September 25, 2026 07:56 Inactive
@frankieroberto
frankieroberto temporarily deployed to record-a-vac-update-vac-tv58vi September 25, 2026 07:59 Inactive

@benjystanton benjystanton left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Have reviewed and thinks it's ready to merge. In the process, I added some urls to the error page links so that they work when clicked.

@benjystanton

Copy link
Copy Markdown
Contributor Author

Reviewed as a team together yesterday, so no need for formal review.

@benjystanton
benjystanton merged commit e1220a3 into main Sep 25, 2026
1 check passed
@benjystanton
benjystanton deleted the update-vaccines-section branch September 25, 2026 08:08

This branch was previously deployed

1 inactive deployment
record-a-vac-update-vac-tv58vi — 7a32994f Deployed Sep 25, 2026 by frankieroberto
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.

3 participants