Skip to content

Add coords to sublet model#52

Merged
alisx255 merged 9 commits into
masterfrom
anthony/backend-geocoords
Mar 27, 2026
Merged

Add coords to sublet model#52
alisx255 merged 9 commits into
masterfrom
anthony/backend-geocoords

Conversation

@alisx255
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Collaborator

@jamesdoh0109 jamesdoh0109 left a comment

Choose a reason for hiding this comment

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

Left comments - after you address them, just make another commit on this branch so this pr stays updated

Comment thread backend/market/models.py Outdated
Comment thread backend/market/models.py Outdated
Comment thread backend/market/models.py Outdated
Comment thread backend/market/models.py Outdated
Comment thread backend/market/models.py Outdated
Comment thread backend/market/serializers.py Outdated
Comment thread backend/market/serializers.py Outdated
Comment thread backend/market/serializers.py Outdated
- Use apporximate latitude/longitude properties instead of calling private methods
- Round coordinates to 6 decimal places in serializer
- Update field names from true_latitude/longitude to latitude/longitude
Comment thread backend/market/models.py Outdated
Comment thread backend/market/models.py Outdated
Comment thread backend/market/models.py Outdated
Comment thread backend/market/models.py
Comment thread backend/market/serializers.py Outdated
Copy link
Copy Markdown
Collaborator

@jamesdoh0109 jamesdoh0109 left a comment

Choose a reason for hiding this comment

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

Good work

@alisx255 alisx255 merged commit 4657130 into master Mar 27, 2026
5 checks passed
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.

2 participants