Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

feat/enterpriseportal: license expiry checker#64317

Merged
bobheadxi merged 6 commits into
mainfrom
08-06-feat_enterpriseportal_license_expiry_checker
Aug 9, 2024
Merged

feat/enterpriseportal: license expiry checker#64317
bobheadxi merged 6 commits into
mainfrom
08-06-feat_enterpriseportal_license_expiry_checker

Conversation

@bobheadxi

@bobheadxi bobheadxi commented Aug 7, 2024 •
edited
Loading

Copy link
Copy Markdown
Member

Port of the existing background job: https://sourcegraph.sourcegraph.com/github.com/sourcegraph/sourcegraph/-/blob/cmd/frontend/internal/dotcom/productsubscription/license_expiration.go?L25:6-25:45, originally added in https://github.com/sourcegraph/sourcegraph/pull/10362

Most of the diff is generated mocks :)

Closes https://linear.app/sourcegraph/issue/CORE-183

Test plan

  • Unit tests
  • Manual test:
[enterprise-...l] INFO service.licenseexpiration.store licenseexpiration/store.go:96 PostToSlack {"text": "The license for subscription `coreservices - Robert Lin - 2024-07-16 23:16:13` <https://sourcegraph.com/site-admin/dotcom/product/subscriptions/bee0a0d9-6c17-4acf-81a4-fd6e777ed6a4|will expire *in the next 24 hours*> :rotating_light:"}

@cla-bot cla-bot Bot added the cla-signed label Aug 7, 2024
@bobheadxi Graphite App

bobheadxi commented Aug 7, 2024 •
edited
Loading

Copy link
Copy Markdown
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @bobheadxi and the rest of your teammates on Graphite Graphite

@bobheadxi bobheadxi mentioned this pull request Aug 7, 2024
Merged
3 tasks
@bobheadxi bobheadxi marked this pull request as ready for review August 7, 2024 00:39
@bobheadxi bobheadxi requested review from a team and dadlerj August 7, 2024 00:39
@bobheadxi bobheadxi force-pushed the 07-26-feat_dotcom_use_enterprise_portal_for_all_subscriptions_ui branch from e99eebb to 8c89de9 Compare August 7, 2024 16:38
@bobheadxi bobheadxi force-pushed the 08-06-feat_enterpriseportal_license_expiry_checker branch from 546a3fe to 587c0e1 Compare August 7, 2024 16:38
@bobheadxi bobheadxi force-pushed the 07-26-feat_dotcom_use_enterprise_portal_for_all_subscriptions_ui branch from 8c89de9 to d72358f Compare August 7, 2024 18:19
@bobheadxi bobheadxi force-pushed the 08-06-feat_enterpriseportal_license_expiry_checker branch from 587c0e1 to 4ff322b Compare August 7, 2024 18:20
@bobheadxi bobheadxi force-pushed the 07-26-feat_dotcom_use_enterprise_portal_for_all_subscriptions_ui branch from d72358f to 3e6bbf6 Compare August 7, 2024 18:26
@bobheadxi bobheadxi force-pushed the 08-06-feat_enterpriseportal_license_expiry_checker branch from 4ff322b to c76dd14 Compare August 7, 2024 18:26
@bobheadxi bobheadxi force-pushed the 07-26-feat_dotcom_use_enterprise_portal_for_all_subscriptions_ui branch from 3e6bbf6 to 374d76d Compare August 7, 2024 18:33
@bobheadxi bobheadxi force-pushed the 08-06-feat_enterpriseportal_license_expiry_checker branch from c76dd14 to af34055 Compare August 7, 2024 18:33
@bobheadxi bobheadxi force-pushed the 07-26-feat_dotcom_use_enterprise_portal_for_all_subscriptions_ui branch from 374d76d to ce72223 Compare August 7, 2024 18:46
@bobheadxi bobheadxi force-pushed the 08-06-feat_enterpriseportal_license_expiry_checker branch from af34055 to 5d99a20 Compare August 7, 2024 18:46
@bobheadxi bobheadxi force-pushed the 07-26-feat_dotcom_use_enterprise_portal_for_all_subscriptions_ui branch from ce72223 to 722b341 Compare August 7, 2024 19:07
@bobheadxi bobheadxi force-pushed the 08-06-feat_enterpriseportal_license_expiry_checker branch 2 times, most recently from 25f1557 to 0290581 Compare August 7, 2024 19:08
@github-actions

github-actions Bot commented Aug 7, 2024

Copy link
Copy Markdown
Contributor

Caution

License checking failed, please read: how to deal with third parties licensing.

1 similar comment
@github-actions

github-actions Bot commented Aug 7, 2024

Copy link
Copy Markdown
Contributor

Caution

License checking failed, please read: how to deal with third parties licensing.

@bobheadxi bobheadxi force-pushed the 07-26-feat_dotcom_use_enterprise_portal_for_all_subscriptions_ui branch from caf37d9 to 9666df5 Compare August 7, 2024 21:15
@bobheadxi bobheadxi force-pushed the 08-06-feat_enterpriseportal_license_expiry_checker branch from 0290581 to 46b6d33 Compare August 7, 2024 21:15
@github-actions

github-actions Bot commented Aug 7, 2024

Copy link
Copy Markdown
Contributor

Caution

License checking failed, please read: how to deal with third parties licensing.

@bobheadxi bobheadxi force-pushed the 07-26-feat_dotcom_use_enterprise_portal_for_all_subscriptions_ui branch from 9666df5 to 82d16dd Compare August 7, 2024 22:27
@bobheadxi bobheadxi force-pushed the 08-06-feat_enterpriseportal_license_expiry_checker branch from 46b6d33 to 7fb030b Compare August 7, 2024 22:27
@github-actions

github-actions Bot commented Aug 7, 2024

Copy link
Copy Markdown
Contributor

Caution

License checking failed, please read: how to deal with third parties licensing.

@bobheadxi bobheadxi force-pushed the 07-26-feat_dotcom_use_enterprise_portal_for_all_subscriptions_ui branch from 82d16dd to 11731a0 Compare August 7, 2024 22:43
@bobheadxi bobheadxi force-pushed the 08-06-feat_enterpriseportal_license_expiry_checker branch from 7fb030b to bef240c Compare August 7, 2024 22:43
@bobheadxi bobheadxi force-pushed the 07-26-feat_dotcom_use_enterprise_portal_for_all_subscriptions_ui branch from 11731a0 to 64ad43c Compare August 7, 2024 22:46
@bobheadxi bobheadxi force-pushed the 08-06-feat_enterpriseportal_license_expiry_checker branch from 2d94ada to b34e194 Compare August 8, 2024 17:39
@bobheadxi bobheadxi force-pushed the 07-25-feat_dotcom_track_cody_gateway_access_directly_in_enterprise_portal branch from 37536df to a5c0a69 Compare August 8, 2024 20:24
@bobheadxi bobheadxi force-pushed the 08-06-feat_enterpriseportal_license_expiry_checker branch from b34e194 to bac27c0 Compare August 8, 2024 20:24
@bobheadxi bobheadxi force-pushed the 07-25-feat_dotcom_track_cody_gateway_access_directly_in_enterprise_portal branch from a5c0a69 to 3f90dcf Compare August 8, 2024 21:35
@bobheadxi bobheadxi force-pushed the 08-06-feat_enterpriseportal_license_expiry_checker branch from bac27c0 to 073ea4d Compare August 8, 2024 21:36
@bobheadxi bobheadxi mentioned this pull request Aug 8, 2024
Merged
@bobheadxi bobheadxi force-pushed the 08-06-feat_enterpriseportal_license_expiry_checker branch from 073ea4d to aa013c8 Compare August 8, 2024 22:08
@bobheadxi bobheadxi force-pushed the 07-25-feat_dotcom_track_cody_gateway_access_directly_in_enterprise_portal branch from 51db89e to 3f90dcf Compare August 8, 2024 22:10
@bobheadxi bobheadxi force-pushed the 08-06-feat_enterpriseportal_license_expiry_checker branch from aa013c8 to b7390d9 Compare August 8, 2024 22:10
@bobheadxi bobheadxi force-pushed the 07-25-feat_dotcom_track_cody_gateway_access_directly_in_enterprise_portal branch from 3f90dcf to f11ce1a Compare August 8, 2024 22:20
@bobheadxi bobheadxi force-pushed the 08-06-feat_enterpriseportal_license_expiry_checker branch 2 times, most recently from 12e4411 to f9f83ff Compare August 8, 2024 22:32
Base automatically changed from 07-25-feat_dotcom_track_cody_gateway_access_directly_in_enterprise_portal to main August 8, 2024 23:37
@bobheadxi bobheadxi force-pushed the 08-06-feat_enterpriseportal_license_expiry_checker branch from f9f83ff to e0c932b Compare August 8, 2024 23:37
jac
jac approved these changes Aug 9, 2024
View reviewed changes

@jac jac left a comment

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.

LGTM!

@bobheadxi bobheadxi force-pushed the 08-06-feat_enterpriseportal_license_expiry_checker branch from e0c932b to e50dd1f Compare August 9, 2024 15:43
@bobheadxi bobheadxi merged commit 43184a3 into main Aug 9, 2024
@bobheadxi bobheadxi deleted the 08-06-feat_enterpriseportal_license_expiry_checker branch August 9, 2024 16:09
This was referenced Aug 9, 2024
Merged
Merged
Closed
Closed
Closed
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@dadlerj dadlerj Awaiting requested review from dadlerj

1 more reviewer

@jac jac jac approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cla-signed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bobheadxi @jac