Skip to content

Exclude duck.ai when using fire button #7152

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Nov 26, 2025
Merged

Exclude duck.ai when using fire button #7152

merged 1 commit into from
Nov 26, 2025

Conversation

@marcosholgado
Copy link
Contributor

@marcosholgado marcosholgado commented Nov 19, 2025 •
edited by malmstein
Loading

Task/Issue URL: https://app.asana.com/1/137249556945/project/1204474923577240/task/1211993499658484?focus=true

Description

Excludes duck.ai when using the fire button

Steps to test this PR

  • Use Duck.ai as normal, adding some chats, upload images, etc
  • Settings → Feature Flag Inventory → Enabled standaloneMigration
  • Close and open the app
  • Go to duck ai
  • You should see a popup asking to refresh, click there
  • Eventually you should see a popup saying "success"
  • You should be taken to duck ai
  • Your chats, history, etc should all be there.
  • Click the back button or back arrow
  • Duck ai WebView should close
  • Use the fire button
  • Go to duck.ai you should not see the refresh popup again and your previous chats should still be available
  • To re-test, clear all the cache and data.

@marcosholgado marcosholgado mentioned this pull request Nov 19, 2025
Merged
13 tasks
@marcosholgado Graphite App
Copy link
Contributor Author

marcosholgado commented Nov 19, 2025 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

@marcosholgado marcosholgado force-pushed the feature/marcos/duckai_exclusions branch from 5566126 to 2216fda Compare November 19, 2025 16:52
@marcosholgado marcosholgado force-pushed the feature/marcos/duckai_migration branch from fff0825 to fbd8f97 Compare November 19, 2025 16:56
@marcosholgado marcosholgado force-pushed the feature/marcos/duckai_exclusions branch from 2216fda to 269b1b0 Compare November 19, 2025 16:56
@marcosholgado marcosholgado force-pushed the feature/marcos/duckai_migration branch from fbd8f97 to 38045a5 Compare November 20, 2025 09:55
@marcosholgado marcosholgado force-pushed the feature/marcos/duckai_exclusions branch 2 times, most recently from cdb6c73 to 68a0681 Compare November 20, 2025 10:45
@marcosholgado marcosholgado force-pushed the feature/marcos/duckai_migration branch 2 times, most recently from b3bb53e to 3a18af1 Compare November 20, 2025 10:59
@marcosholgado marcosholgado force-pushed the feature/marcos/duckai_exclusions branch from 68a0681 to 4987961 Compare November 20, 2025 11:25
@marcosholgado marcosholgado force-pushed the feature/marcos/duckai_migration branch from 3a18af1 to 10ec9f3 Compare November 25, 2025 10:04
@marcosholgado marcosholgado force-pushed the feature/marcos/duckai_exclusions branch from 4987961 to a643268 Compare November 25, 2025 10:04
malmstein
malmstein approved these changes Nov 25, 2025
View reviewed changes
Copy link
Contributor

@malmstein malmstein left a comment

Choose a reason for hiding this comment

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

Works as expected. :shipit:

joshliebe
joshliebe approved these changes Nov 26, 2025
View reviewed changes
Copy link
Contributor

@joshliebe joshliebe left a comment

Choose a reason for hiding this comment

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

LGTM

Base automatically changed from feature/marcos/duckai_migration to develop November 26, 2025 13:57
@marcosholgado marcosholgado force-pushed the feature/marcos/duckai_exclusions branch from a643268 to 48e6099 Compare November 26, 2025 14:01
@marcosholgado marcosholgado merged commit 5aff230 into develop Nov 26, 2025
10 of 11 checks passed
@marcosholgado marcosholgado deleted the feature/marcos/duckai_exclusions branch November 26, 2025 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@malmstein malmstein malmstein approved these changes

@joshliebe joshliebe joshliebe approved these changes

Assignees

@malmstein malmstein

@joshliebe joshliebe

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@marcosholgado @malmstein @joshliebe