Open
Conversation
- Add fallback sender email address (noreply@fashionunited.com) - Use repository variables (vars) instead of secrets for non-sensitive config - Add better error debugging to show response body on failure - Make email failures non-blocking (workflow continues even if email fails) - Add logging for sender email and name for debugging The 400 error was caused by missing BREVO_SENDER_EMAIL configuration. 🤖 Generated with Claude Code (https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Add comprehensive documentation for browsing the SQLite database including: - Basic database access commands - Table structure viewing - Company information lookup examples (using MYTE as example) - Useful queries for market cap analysis - CSV export examples 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Prefix unused struct fields with underscore to eliminate compilation warnings for fields that are needed for CSV deserialization but not used in the code: - _absolute_change - _market_share_from - _market_share_to 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Implement comprehensive system to track and apply ticker symbol changes from mergers, acquisitions, and rebranding events. Features: - New CLI commands: check-symbol-changes and apply-symbol-changes - Fetch symbol changes from Financial Modeling Prep API - Identify which changes apply to configured tickers - Automatic config.toml updates with backup creation - Database tracking to prevent reapplying changes - Dry-run mode for previewing changes - Detailed reporting of applicable/non-applicable changes Database: - Add symbol_changes table to track all changes - Store old/new symbols, change dates, company names - Track application status to avoid duplicates Safety: - Automatic backup before applying changes - Conflict detection for duplicate symbols - Comments added showing old ticker and change date 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Gap Inc. changed ticker from GPS to GAP on 2024-08-22. Remove the old GPS ticker to avoid conflicts. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Mytheresa changed ticker from MYTE to LUXE on 2025-05-01. Also marked this change and the GPS->GAP change as applied in database. Other symbol changes from API appear to be outdated/incorrect: - UA, REAL, LANV, TPR, CHS are still the correct current tickers - The suggested changes to UAA, RLBY, STRM, PLNTQ, CHBD have no market data 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Split workflow into separate build, collect-data, compare-data, and visualize jobs - Pass database between jobs using artifacts for data continuity - Add automatic month-over-month comparison (today vs last month) - Handle edge cases for month-end dates (e.g., 31st to Feb 28/29) - Generate visualization charts from comparison data - Improve email notifications with detailed job status reporting The workflow now: 1. Builds the binary once and reuses across jobs 2. Collects today's market data 3. Compares with last month's data (handles month-end edge cases) 4. Generates visualization charts 5. Sends comprehensive status notification 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.