Skip to content

feat: make brief public#2965

Merged
idoshamun merged 1 commit intomainfrom
brief-public
Aug 6, 2025
Merged

feat: make brief public#2965
idoshamun merged 1 commit intomainfrom
brief-public

Conversation

@idoshamun
Copy link
Member

To make brief shareable we decided to make them public since there's no private/personal info there

@pulumi
Copy link

pulumi bot commented Aug 6, 2025

🍹 The Update (preview) for dailydotdev/api/prod (at fa4f284) was successful.

Resource Changes

    Name                                            Type                           Operation
~   vpc-native-check-analytics-report-cron          kubernetes:batch/v1:CronJob    update
~   vpc-native-update-tag-recommendations-cron      kubernetes:batch/v1:CronJob    update
~   vpc-native-deployment                           kubernetes:apps/v1:Deployment  update
~   vpc-native-clean-stale-user-transactions-cron   kubernetes:batch/v1:CronJob    update
~   vpc-native-private-deployment                   kubernetes:apps/v1:Deployment  update
~   vpc-native-update-trending-cron                 kubernetes:batch/v1:CronJob    update
~   vpc-native-update-views-cron                    kubernetes:batch/v1:CronJob    update
~   vpc-native-clean-zombie-users-cron              kubernetes:batch/v1:CronJob    update
~   vpc-native-calculate-top-readers-cron           kubernetes:batch/v1:CronJob    update
~   vpc-native-personalized-digest-deployment       kubernetes:apps/v1:Deployment  update
~   vpc-native-clean-gifted-plus-cron               kubernetes:batch/v1:CronJob    update
~   vpc-native-generate-search-invites-cron         kubernetes:batch/v1:CronJob    update
~   vpc-native-sync-subscription-with-cio-cron      kubernetes:batch/v1:CronJob    update
~   vpc-native-update-current-streak-cron           kubernetes:batch/v1:CronJob    update
~   vpc-native-update-source-tag-view-cron          kubernetes:batch/v1:CronJob    update
~   vpc-native-bg-deployment                        kubernetes:apps/v1:Deployment  update
~   vpc-native-ws-deployment                        kubernetes:apps/v1:Deployment  update
~   vpc-native-clean-zombie-images-cron             kubernetes:batch/v1:CronJob    update
-   vpc-native-api-migration-762b68c6               kubernetes:batch/v1:Job        delete
+   vpc-native-api-migration-419bf488               kubernetes:batch/v1:Job        create
~   vpc-native-update-source-public-threshold-cron  kubernetes:batch/v1:CronJob    update
~   vpc-native-clean-zombie-user-companies-cron     kubernetes:batch/v1:CronJob    update
~   vpc-native-temporal-deployment                  kubernetes:apps/v1:Deployment  update
~   vpc-native-hourly-notification-cron             kubernetes:batch/v1:CronJob    update
~   vpc-native-personalized-digest-cron             kubernetes:batch/v1:CronJob    update
~   vpc-native-daily-digest-cron                    kubernetes:batch/v1:CronJob    update
~   vpc-native-update-tags-str-cron                 kubernetes:batch/v1:CronJob    update
~   vpc-native-generic-referral-reminder-cron       kubernetes:batch/v1:CronJob    update
~   vpc-native-validate-active-users-cron           kubernetes:batch/v1:CronJob    update
~   vpc-native-update-highlighted-views-cron        kubernetes:batch/v1:CronJob    update

Copy link
Contributor

@omBratteng omBratteng left a comment

Choose a reason for hiding this comment

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

Just a semantic fix really? Because I've been able to share briefs already 🙈

Example https://app.daily.dev/posts/aug-6-2025-qabg1jqmx

@idoshamun
Copy link
Member Author

@omBratteng i can't see the content, just the title

@omBratteng
Copy link
Contributor

@omBratteng i can't see the content, just the title

I've been testing in incognito :lolsob:, there I can view everything...

Logged in Anonymous
image image

@idoshamun
Copy link
Member Author

Maybe it's a bug for logged in users 🤷

@idoshamun idoshamun merged commit 7ec270f into main Aug 6, 2025
10 checks passed
@idoshamun idoshamun deleted the brief-public branch August 6, 2025 16:03
@capJavert
Copy link
Contributor

I discovered it earlier in the morning for anon, merged hour ago #2964

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