Skip to content

fix: first time deployment dialog not showing #7194

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.

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AdityaHegde
Copy link
Collaborator

closes https://github.com/rilldata/rill-private-issues/issues/1322

We need to show the first time deployment to new users. But new users are most likely to not be logged in or registered even. So as a hack we also show the message to users not logged in.

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

Copy link

Repository not enabled for automatic review

The Bito agent is set up, but this repository is not enabled for automatic code reviews. To review this pull request, type /review in a comment and save it.

You can enable automatic reviews in the agent setup here or contact your Bito workspace admin at [email protected].

Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers a full AI review.

Refer to the documentation for additional commands.

Configuration
This repository uses Default Agent You can customize the agent settings here or contact your Bito workspace admin at [email protected].

Documentation & Help

@AdityaHegde AdityaHegde requested a review from ericpgreen2 April 25, 2025 10:34
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.

1 participant