Skip to content

fix(config): rename billing.config.js to billing.development.config.js #3895

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
PierreBrisorgueil merged 2 commits into from
Apr 7, 2026
Merged

fix(config): rename billing.config.js to billing.development.config.js #3895

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
394efaa
fix(config): rename billing.config.js to billing.development.config.js
PierreBrisorgueil Apr 7, 2026
162689d
fix(billing): use named import for plans, add migration note for bill…
PierreBrisorgueil Apr 7, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 13 additions & 0 deletions MIGRATIONS.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,19 @@ To adopt the pattern:

---

## Billing config rename (2026-04-07)

`billing.config.js` has been renamed to `billing.development.config.js` to align with the `{module}.development.config.js` stack convention.

### Action for downstream

If your downstream project imports or overrides `billing.config.js`, rename it:

- `src/modules/billing/config/billing.config.js` → `src/modules/billing/config/billing.development.config.js`
- Update any `import` statements from `'../config/billing.config'` to `'../config/billing.development.config'`

---

## Module Activation Config (2026-04-05)

Per-module `activated: true/false` config flag. When `activated: false`, the module's routes are not mounted and it's invisible to the app.
Expand Down
0 src/modules/billing/config/billing.config.js → ...ling/config/billing.development.config.js
View file
Open in desktop
File renamed without changes.
2 changes: 1 addition & 1 deletion src/modules/billing/views/billing.pricing.view.vue
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@
*/
import { useBillingStore } from '../stores/billing.store';
import { useAuthStore } from '../../auth/stores/auth.store';
import plansConfig from '../config/billing.config';
import { plans as plansConfig } from '../config/billing.development.config';
import billingPricingToggleComponent from '../components/billing.pricingToggle.component.vue';
Comment on lines 87 to 90

Copilot AI Apr 7, 2026

Copy link

Choose a reason for hiding this comment

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

Issue #3892 mentions adding a MIGRATIONS.md note for downstream projects about the billing.config.js → billing.development.config.js rename, but this PR doesn’t include any migration documentation update (and MIGRATIONS.md currently has no billing-specific entry). Please add an explicit migration entry so downstream overrides/imports can be updated reliably.

Copilot uses AI. Check for mistakes.

@PierreBrisorgueil PierreBrisorgueil Apr 7, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Fixed in 2741b5d — added a 'Billing config rename' section to MIGRATIONS.md documenting the rename and the import update required for downstream projects.

import billingPricingCardComponent from '../components/billing.pricingCard.component.vue';

Expand Down
Loading