Skip to content

Comments

fix(slack): clean up footer branding and view URL#301

Merged
Selinali01 merged 2 commits intomainfrom
fix/slack-footer-text
Feb 10, 2026
Merged

fix(slack): clean up footer branding and view URL#301
Selinali01 merged 2 commits intomainfrom
fix/slack-footer-text

Conversation

@Selinali01
Copy link
Contributor

@Selinali01 Selinali01 commented Feb 10, 2026

Simplify the view URL by removing executionId query param and fix branding text ("BubbleLab" -> "Bubble Lab", "View execution" -> "View Flow").

Summary

Related Issues

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Refactor
  • New Bubble Integration
  • Other (please describe):

Checklist

  • My code follows the code style of this project
  • I have added appropriate tests for my changes
  • I have run pnpm check and all tests pass
  • I have tested my changes locally
  • I have linked relevant issues

Screenshots (Required)

For New Bubble Integrations

📋 Integration Flow Tests: When creating a new bubble, you must write an integration flow test that exercises all operations end-to-end in realistic scenarios—including edge cases. This flow should be runnable in bubble studio and return structured results tracking each operation's success/failure with details. See packages/bubble-core/src/bubbles/service-bubble/google-sheets/google-sheets.integration.flow.ts for a complete reference implementation.

⚠️ If your integration requires API credits for testing, please reach out to the team for test credentials.

  • Integration flow test (.integration.flow.ts) covers all operations
  • Screenshots showing full test results in Bubble Studio attached above

Additional Context

Summary by CodeRabbit

  • Updates

    • Slack integration footer now displays "View Flow" with updated branding ("Bubble Lab") for improved clarity.
  • Chores

    • Version bump to 0.1.142 across packages.

Simplify the view URL by removing executionId query param and fix
branding text ("BubbleLab" -> "Bubble Lab", "View execution" -> "View Flow").

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link

coderabbitai bot commented Feb 10, 2026

📝 Walkthrough

Walkthrough

This pull request applies a patch version bump (0.1.141 → 0.1.142) across five packages in the monorepo and updates the Slack footer within the bubble-core service to remove execution context from the view URL while updating associated footer text and branding labels.

Changes

Cohort / File(s) Summary
Package Version Bumps
packages/bubble-core/package.json, packages/bubble-runtime/package.json, packages/bubble-scope-manager/package.json, packages/bubble-shared-schemas/package.json, packages/create-bubblelab-app/package.json
Incremented version from 0.1.141 to 0.1.142 across all packages.
Slack Footer Updates
packages/bubble-core/src/bubbles/service-bubble/slack/slack.ts
Modified Slack footer construction: removed executionId query parameter from view URL, changed footer text from "View execution" to "View Flow", and updated branding label from "BubbleLab" to "Bubble Lab".

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • Metadata for slack bubbles #297: Directly related through Slack footer construction logic modifications affecting the same footer metadata and view link handling.
  • bump version #293: Related through coordinated package.json version bumps across the same set of packages.
  • bump versions #286: Related through version field updates in shared packages like bubble-core and bubble-runtime.

Poem

🐰 Hop, hop! A version's born anew,
0.1.142 shines through,
Slack footer now says "Flow" with care,
Bubble Lab branding everywhere!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix(slack): clean up footer branding and view URL' directly and accurately summarizes the main changes in the PR, which involve updating Slack footer branding text, removing the executionId parameter from the view URL, and updating button text.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/slack-footer-text

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
packages/bubble-core/src/bubbles/service-bubble/slack/slack.ts (1)

1503-1509: Stale comment and unused parameter type field.

The JSDoc on line 1503 still says "Powered by BubbleLab" (same at line 1561). Also, executionId in the parameter type (line 1508) is no longer referenced in the method body after removing it from the URL. Consider updating for consistency.

Suggested cleanup
   /**
-   * Build "Powered by BubbleLab" footer blocks from execution metadata.
+   * Build "Powered by Bubble Lab" footer blocks from execution metadata.
    * Returns empty array if no metadata is available.
    */
   private static buildFooterBlocks(executionMeta?: {
     flowId?: number;
-    executionId?: number;
     studioBaseUrl?: string;
   }): Record<string, unknown>[] {

And at line 1561:

-    // Build "Powered by BubbleLab" footer from execution metadata
+    // Build "Powered by Bubble Lab" footer from execution metadata

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Feb 10, 2026

Deploying bubblelab-documentation with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1f3cf01
Status: ✅  Deploy successful!
Preview URL: https://18cd275b.bubblelab-documentation.pages.dev
Branch Preview URL: https://fix-slack-footer-text.bubblelab-documentation.pages.dev

View logs

@Selinali01 Selinali01 merged commit c8e6ec4 into main Feb 10, 2026
8 checks passed
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