Skip to content

feat: add fixtures#2641

Merged
JacobCoffee merged 7 commits into
mainfrom
bootstrap-sponsors-in-dev
Feb 19, 2026
Merged

feat: add fixtures#2641
JacobCoffee merged 7 commits into
mainfrom
bootstrap-sponsors-in-dev

Conversation

@JacobCoffee
Copy link
Copy Markdown
Member

@JacobCoffee JacobCoffee commented Oct 8, 2024

Description

Add fixture files for bootstrapping development environments with sample data:

  • fixtures/sponsors-admin.json - Sponsor application data for the admin user
  • fixtures/events.json - Sample events, calendars, and categories

Changes from review feedback

  • Rebased on latest main
  • Converted all fixtures to use --natural-foreign / --natural-primary key format per @ewdurbin's review
    • ContentType references use ["app_label", "model"] instead of numeric IDs
    • User references use ["username"] instead of numeric PKs
    • This prevents fixture breakage when content type IDs change
  • Squashed into a single commit

@JacobCoffee JacobCoffee requested a review from ewdurbin as a code owner October 8, 2024 18:42
@JacobCoffee JacobCoffee changed the title feat: add admin user sponsor app generated in dev feat: add fixtures Oct 8, 2024
Comment thread fixtures/sponsors-admin.json Outdated
Add fixture files for bootstrapping development environments with
sample sponsor and event data. Uses natural foreign keys for
content types and user references to avoid breakage when content
type IDs change.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings February 18, 2026 23:53
@JacobCoffee JacobCoffee force-pushed the bootstrap-sponsors-in-dev branch from e673822 to ec4c80c Compare February 18, 2026 23:53
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds new Django fixture files intended to seed development environments with example Sponsors data (including an “admin sponsor” scenario) and Events data.

Changes:

  • Added fixtures/sponsors-admin.json with a large set of Sponsors-related objects for dev seeding.
  • Added fixtures/events.json with a minimal events calendar/category/location/events + rules dataset.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 3 comments.

File Description
fixtures/sponsors-admin.json New sponsors fixture intended to preload a sample sponsor/sponsorship dataset (and related assets/config).
fixtures/events.json New events fixture intended to preload a sample calendar and a few sample events/rules.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread fixtures/sponsors-admin.json
Comment thread fixtures/sponsors-admin.json Outdated
Comment thread fixtures/sponsors-admin.json Outdated
JacobCoffee and others added 2 commits February 18, 2026 18:30
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings February 19, 2026 00:31
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 2 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread fixtures/events.json
Comment thread fixtures/sponsors-admin.json Outdated
Comment thread fixtures/sponsors-admin.json
JacobCoffee and others added 3 commits February 18, 2026 18:55
The docs dependencies are in [dependency-groups], not
[project.optional-dependencies], so pip install '.[docs]' fails.
Use pip install --group docs to match the pattern from #2902.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- "comlimentary" → "complimentary" in internal_name
- "retreival" → "retrieval" in internal_name
- "sponsorshp" → "sponsorship" in description

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings February 19, 2026 01:02
@JacobCoffee JacobCoffee merged commit 863dedc into main Feb 19, 2026
15 of 16 checks passed
@JacobCoffee JacobCoffee deleted the bootstrap-sponsors-in-dev branch February 19, 2026 01:02
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .readthedocs.yaml
Comment on lines 10 to 12
python: "3"

commands:
Copy link

Copilot AI Feb 19, 2026

Choose a reason for hiding this comment

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

Read the Docs’ Ubuntu 22.04 image may ship with an older pip that doesn’t support pip install --group ..., which would make the docs build fail. Consider explicitly upgrading pip (and optionally setuptools/wheel) before this command, and/or pinning build.tools.python to 3.12 to match requires-python = ">=3.12" in pyproject.toml.

Suggested change
python: "3"
commands:
python: "3.12"
commands:
- python -m pip install --upgrade pip setuptools wheel

Copilot uses AI. Check for mistakes.
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