Skip to content

Fix app with service setting install bug#1284

Open
jakeymac wants to merge 3 commits into
mainfrom
fix-app-with-services-install
Open

Fix app with service setting install bug#1284
jakeymac wants to merge 3 commits into
mainfrom
fix-app-with-services-install

Conversation

@jakeymac

@jakeymac jakeymac commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Description

This merge request addresses a bug where when installing an app that has a service setting, but there are no existing services, an error would show up.

Changes Made to Code

  • Updated the run_interactive_services function to not just look for the first service query ([0]), but to flatten out the query(ies) to avoid errors and to make sure it's checking for multiple query sets such as in the case of persistent store services

Related PRs, Issues, and Discussions

Quality Checks

  • At least one new test has been written for new code
  • New code has 100% test coverage
  • Code has been formatted with Black
  • Code has been linted with flake8
  • Docstrings for new methods have been added
  • The documentation has been updated appropriately

@jakeymac jakeymac self-assigned this Jul 8, 2026
@coveralls

coveralls commented Jul 8, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 100.0%. remained the same — fix-app-with-services-install into main

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.

2 participants