Skip to content

Feat/addon logs#887

Closed
miton18 wants to merge 1 commit intomasterfrom
feat/addon-logs
Closed

Feat/addon logs#887
miton18 wants to merge 1 commit intomasterfrom
feat/addon-logs

Conversation

@miton18
Copy link
Copy Markdown
Contributor

@miton18 miton18 commented Feb 25, 2025

TODO:

  • update client and use it
  • add feature flag (default: stack v2, enabled stack v4)

@miton18 miton18 requested a review from a team as a code owner February 25, 2025 11:01
@hsablonniere hsablonniere added this to the 3.13 milestone Feb 26, 2025
@miton18 miton18 force-pushed the feat/addon-logs branch 3 times, most recently from f289aa1 to 405c96b Compare March 11, 2025 14:50
@hsablonniere hsablonniere modified the milestone: 3.13 May 22, 2025
@miton18 miton18 force-pushed the feat/addon-logs branch from 405c96b to ffeff3e Compare May 26, 2025 10:03
@hsablonniere hsablonniere modified the milestones: 3.13, 3.14 May 27, 2025
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Sep 30, 2025

🔎 A preview has been automatically published!

If you created the alias to the preview script, you can run this command to download and install this preview:

clever-preview update feat-addon-logs

You can also run it from your local repository:

./scripts/preview.js update feat-addon-logs
OS SHA256 checksum
🐧 linux 09f107f6ce881b44fec5efad085d370e31617ad312a4c3f032ebd052d0de72bf
🍏 macos 013771a3932a13e3af57f145e49b9a2954add8bd11abbd41b076056d54725093

This preview will be deleted once this PR is closed.

@miton18 miton18 force-pushed the feat/addon-logs branch 2 times, most recently from d1d370e to 0e441bd Compare September 30, 2025 22:02
Comment thread src/commands/logs.js
Comment on lines +26 to +27
const addonId = await resolveRealId(addonIdOrRealId);
const ownerId = await resolveOwnerId(addonId);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Maybe findAddonsByNameOrId() can help here

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We never supported add-on names in this commande. Let's stick to IDs for now.

@hsablonniere hsablonniere modified the milestones: 4.4.0, 4.5.0 Nov 12, 2025
@hsablonniere hsablonniere removed this from the 4.5.0 milestone Dec 8, 2025
@hsablonniere hsablonniere deleted the feat/addon-logs branch January 9, 2026 21:57
@hsablonniere hsablonniere restored the feat/addon-logs branch January 9, 2026 22:00
@hsablonniere hsablonniere reopened this Jan 9, 2026
@miton18
Copy link
Copy Markdown
Contributor Author

miton18 commented Jan 20, 2026

up

@StevenLeRoux
Copy link
Copy Markdown

@miton18 do you need the second item (feature flag) to be complete before merging it?

@hsablonniere
Copy link
Copy Markdown
Member

Closing in favor of #1032

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.

4 participants