Skip to content

Conversation

@TheStrgamer
Copy link
Contributor

@TheStrgamer TheStrgamer commented Feb 27, 2025

Proposed changes

Made it so quotes that have the same date are sorted by descending id.

Checklist

(If any of the points are not relevant, mark them as checked)

  • Remembered to run the makemigrations, makemessages and compilemessages management commands and committed any changes that should be included in this PR
  • Created tests that fail without the changes, if relevant/possible
  • Manually tested that the website UI works as intended with different device layouts
    • (Most common is to test with typical screen sizes for mobile (320-425 px), tablet (768 px) and desktop (1024+ px), which can easily be done with your browser's dev tools)
  • Manually tested that everything works as intended when logged in as different users locally
    • (This can be e.g. anonymous users (i.e. not being logged in), "normal" non-member users, members of different committees, and superusers)
  • Made sure that your code conforms to the code style guides
    • (It's not intended that you read through this whole document, but that you get yourself an overview over its contents, and that you use it as a reference guide / checklist while taking a second look at your code before opening a pull request)
  • Attempted to minimize the number of common code smells
    • (See the comment for the previous checkbox)
  • Added sufficient documentation - e.g. as comments, docstrings or in the README, if suitable
  • Added your changes to the "Unreleased" section of the changelog - mainly the changes that are of particular interest to users and/or developers, if any
  • Added a "Deployment notes" section above, if anything out of the ordinary should be done when deploying these changes to the server
  • Structured your commits reasonably

@make-bot make-bot bot added this to web Feb 27, 2025
@make-bot make-bot bot moved this to Ready for Review in web Feb 27, 2025
@TheStrgamer TheStrgamer changed the base branch from main to dev February 27, 2025 18:49
@codecov
Copy link

codecov bot commented Feb 27, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.80%. Comparing base (2f2af32) to head (d624c34).
⚠️ Report is 12 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #754      +/-   ##
==========================================
- Coverage   88.28%   87.80%   -0.49%     
==========================================
  Files         152      153       +1     
  Lines        6207     6241      +34     
==========================================
  Hits         5480     5480              
- Misses        727      761      +34     
Files with missing lines Coverage Δ
src/internal/forms.py 87.15% <100.00%> (ø)
src/internal/models.py 93.78% <100.00%> (ø)
src/internal/views.py 90.90% <100.00%> (ø)

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ddabble ddabble added the ⚡Quick review⚡ This PR is short and can be reviewed quickly. label Mar 23, 2025
@ddabble ddabble deleted the branch main September 7, 2025 19:48
@ddabble ddabble closed this Sep 7, 2025
@github-project-automation github-project-automation bot moved this from Ready for Review to Done in web Sep 7, 2025
@ddabble ddabble reopened this Sep 7, 2025
@ddabble ddabble changed the base branch from dev to main September 7, 2025 20:01
@make-bot make-bot bot moved this from Done to Ready for Review in web Sep 7, 2025
@ddabble
Copy link
Member

ddabble commented Sep 7, 2025

Did an oopsie - see #758 for context :)

Copy link
Member

@ddabble ddabble left a comment

Choose a reason for hiding this comment

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

Looks good! 😊

migrations.AlterField(
model_name='quote',
name='time',
field=models.DateTimeField(verbose_name='-time it was said'),
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
field=models.DateTimeField(verbose_name='-time it was said'),
field=models.DateTimeField(verbose_name='time it was said'),

Copy link
Member

@ddabble ddabble left a comment

Choose a reason for hiding this comment

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

Ah, I forgot: I think you should update the changelog with this fix :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚡Quick review⚡ This PR is short and can be reviewed quickly.

Projects

Status: Ready for Review

Development

Successfully merging this pull request may close these issues.

3 participants