Skip to content

Conversation

maryhipp
Copy link
Collaborator

What type of PR is this? (check all applicable)

  • Refactor
  • [X ] Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Have you discussed this change with the InvokeAI team?

  • [X ] Yes
  • No, because:

Description

Adds route and settings UI to clear 100 intermediates at a time

Related Tickets & Documents

  • Related Issue #
  • Closes #

QA Instructions, Screenshots, Recordings

Added/updated tests?

  • Yes
  • No : please replace this line with details on why tests
    have not been included

[optional] Are there any post deployment tasks we need to perform?

Copy link
Contributor

@brandonrising brandonrising left a comment

Choose a reason for hiding this comment

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

lgtm

@maryhipp maryhipp enabled auto-merge (squash) July 19, 2023 14:25
Copy link
Collaborator

@psychedelicious psychedelicious left a comment

Choose a reason for hiding this comment

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

I'm happy to approve this for now.

If there is time for the suggested changes, great. If not, can you please make an issue to track them?

@maryhipp maryhipp disabled auto-merge July 19, 2023 14:32
@maryhipp maryhipp enabled auto-merge (squash) July 19, 2023 14:43
@maryhipp maryhipp merged commit 9f00e05 into main Jul 19, 2023
@maryhipp maryhipp deleted the maryhipp/clear-intermediates branch July 19, 2023 14:55
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.

4 participants