Skip to content
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

Support firebase dataconnect:sdk:generate --watch #7719

Merged
merged 4 commits into from
Sep 24, 2024
Merged

Conversation

rosalyntan
Copy link
Member

Description

Scenarios Tested

Sample Commands

@rosalyntan rosalyntan changed the title Support firebase dataconnect:sdk:generaate --watch Sep 23, 2024
@@ -8,9 +8,11 @@ import { load } from "../dataconnect/load";
import { readFirebaseJson } from "../dataconnect/fileUtils";
import { logger } from "../logger";

type GenerateOptions = Options & { watch?: boolean };
Copy link
Contributor

@fredzqm fredzqm Sep 23, 2024

Choose a reason for hiding this comment

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

[Can be defered] [Not blocking PP, lower priority than compatible modes docs]

While we are at this. It would nice if sdk:generate also takes optional [serviceId] [connectorId] like other commands.

The rule of thumb from go/fdc:multi-data-source: if there are >1 services / connectors / datasource, it defaults to run the same routine against all relevant resources.

  dataconnect:services:list                                      list all deployed services in your Firebase 
                                                                 project
  dataconnect:sql:diff [serviceId]                               displays the differences between  a local 
                                                                 DataConnect schema and your CloudSQL database's 
                                                                 current schema
  dataconnect:sql:migrate [options] [serviceId]                  migrates your CloudSQL database's schema to 
                                                                 match your local DataConnect schema
  dataconnect:sql:grant [options] [serviceId]                    Grants the SQL role <role> to the provided user 
                                                                 or service account <email>.
  dataconnect:sdk:generate                                       generates typed SDKs for your Data Connect 
                                                                 connectors
Copy link
Member Author

Choose a reason for hiding this comment

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

Filed b/369400626 to update all commands per go/fdc:multi-data-source, as I believe right now an error is thrown in diff/migrate/grant if serviceId isn't specified and multiple services exist.

@rosalyntan rosalyntan marked this pull request as ready for review September 24, 2024 19:12
@rosalyntan rosalyntan requested a review from joehan September 24, 2024 19:12
@rosalyntan rosalyntan merged commit 35d16be into master Sep 24, 2024
40 of 41 checks passed
@rosalyntan rosalyntan deleted the rosalyntan.watch branch September 24, 2024 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants