Skip to content

Conversation

@HarshMN2345
Copy link
Member

@HarshMN2345 HarshMN2345 commented Nov 4, 2025

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

image image

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

Summary by CodeRabbit

  • Updates
    • Updated promotional announcement displayed in the cloud console. The new promotion highlights database operators functionality with refreshed messaging, updated visual assets, and a new call-to-action link directing users to relevant documentation.

@appwrite
Copy link

appwrite bot commented Nov 4, 2025

Console

Project ID: 688b7bf400350cbd60e9

Sites (1)
Site Status Logs Preview QR
 console-stage
688b7cf6003b1842c9dc
Building Building View Logs Preview URL QR Code

Tip

Cursor pagination performs better than offset pagination when loading further pages.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 4, 2025

Walkthrough

This pull request updates the promotional alert displayed in the console bottom alerts. When isCloud is true, the system now displays a DB operators announcement instead of a Transactions API promo. The changes include renaming the promo object from transactionsApiPromo to dbOperatorsPromo, updating promotional metadata (id, title, message), swapping image assets from transactions-api to db-operators variants, and updating the CTA link to point to the DB operators blog post instead of the transactions API blog post.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

  • Single file with straightforward promotional content replacement
  • Consistent pattern swap with no logic modifications
  • Verify image asset paths are correctly updated and files exist
  • Confirm CTA link URL is valid and points to correct destination

Pre-merge checks and finishing touches

✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: introducing a new console notification feature for DB operators, which aligns with the code changes that replace the transactions API promo with a DB operators promo.
✨ 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 feat-SER-517-Console-Notification-DB-operators

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.

@HarshMN2345 HarshMN2345 requested a review from Meldiron November 4, 2025 06:29
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c5fd02a and b823f12.

⛔ Files ignored due to path filters (4)
  • src/lib/images/promos/db-operators-dark.png is excluded by !**/*.png
  • src/lib/images/promos/db-operators-light.png is excluded by !**/*.png
  • src/lib/images/promos/transactions-api-dark.png is excluded by !**/*.png
  • src/lib/images/promos/transactions-api-light.png is excluded by !**/*.png
📒 Files selected for processing (1)
  • src/routes/(console)/bottomAlerts.ts (1 hunks)
🧰 Additional context used
🧬 Code graph analysis (1)
src/routes/(console)/bottomAlerts.ts (1)
src/lib/system.ts (1)
  • isCloud (26-26)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: build
  • GitHub Check: e2e
🔇 Additional comments (2)
src/routes/(console)/bottomAlerts.ts (2)

10-29: LGTM! Clean implementation of the promotional content swap.

The changes correctly replace the Transactions API promo with the DB operators promo. All references are consistently updated, including:

  • Variable naming (dbOperatorsPromo)
  • Promotional ID (modal:db_operators_announcement)
  • Title, message, and CTA link
  • Image asset references

The implementation maintains the same structure and logic flow, making this a straightforward content update.


4-5: Verification complete—image assets confirmed to exist and old promotional content fully removed.

The new image imports at lines 4-5 reference existing files (src/lib/images/promos/db-operators-dark.png and db-operators-light.png). The old transaction API promotional content has been completely removed from the codebase with no lingering references. The DB operators promo is properly scoped within the isCloud conditional and follows the expected structure.

@ItzNotABug ItzNotABug merged commit 20e5930 into main Nov 4, 2025
3 of 4 checks passed
@ItzNotABug ItzNotABug deleted the feat-SER-517-Console-Notification-DB-operators branch November 4, 2025 13:52
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.

3 participants