Skip to content
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

Jump to bottom

Misc adjustments #357

Merged
merged 7 commits into from
Oct 3, 2024
Merged

Misc adjustments #357

merged 7 commits into from
Oct 3, 2024

Conversation

birbjam
Copy link
Contributor

@birbjam birbjam commented Oct 2, 2024

In this PR:

  • #868a2c37t: Showing Project Status table in admin under Helpers section.
  • #8689qebva: Removed Location: None specified placeholders from dashcards -- get_location() now returns None instead, no placeholder appears for location if none is added in settings or on account creation.
  • Adjusted required and not required fields for account models and forms in the settings.
  • Changed Inquiry Type to Subscription Type field name on account creation.
  • Removed some unused imports.

@arojas1
Copy link
Contributor

arojas1 commented Oct 2, 2024

Task linked: CU-8689qebva Remove Location placeholder from dashcards
Task linked: CU-868a2c37t Admin: show Project statuses

@birbjam birbjam marked this pull request as ready for review October 3, 2024 20:22
@birbjam birbjam merged commit e4b10a2 into subscriptions Oct 3, 2024
1 check passed
@birbjam birbjam deleted the misc-adjustments branch October 3, 2024 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@birbjam @arojas1