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

iD editor upgrade to v2.30.2 #119

Merged
merged 214 commits into from
Dec 18, 2024
Merged

iD editor upgrade to v2.30.2 #119

merged 214 commits into from
Dec 18, 2024

Conversation

royallsilwallz
Copy link

What type of PR is this? (check all applicable)

  • 🍕 Feature
  • 🐛 Bug Fix
  • 📝 Documentation
  • 🧑‍💻 Refactor
  • ✅ Test
  • 🤖 Build or CI
  • ❓ Other (please specify)

Related Issue

Example: Fixes #123

Describe this PR

A brief description of how this solves the issue.

Screenshots

Please provide screenshots of the change.

Alternative Approaches Considered

Did you attempt any other approaches that are not documented in code?

Review Guide

Notes for the reviewer. How to test this change?

Checklist before requesting a review

[optional] What gif best describes this PR or how it makes you feel?

spwoodcock and others added 30 commits April 30, 2024 17:42
@spwoodcock
ci: use hotosm/gh-workflows for container img builds
0579892
@mahesh-naxa
ref: frontend tests now has parallel workers & removed warnings
002384e
@eternaltyro
Cleanup backend dockerfile
74a705a 
- Upgrade pdm and pydebug versions
- Reduce RUN layers for apt steps
- Remove gunicorn workers parameter to allow for WEB_CONCURRENCY env

Signed-off-by: eternaltyro <230743+eternaltyro@users.noreply.github.com>
@mahesh-naxa @dakotabenjamin
add: general pre-commit hooks for static code checks
70f5bca
@mahesh-naxa @dakotabenjamin
pre-commit: run --all-files applied
85ee336
@mahesh-naxa @dakotabenjamin
applied pre-commit run-all files for end-of-file-fixer & trailing-whi…
26413e1 
…tespace
@royallsilwallz
Remove Footer in Create New Partner page
b1d01c5
@royallsilwallz
Remove grain background in Manage Partners section
07ec188
@royallsilwallz
Do not fetch data if current projects is null in Partners Page
cd3b1bc
@royallsilwallz
Add no current projects placeholder text for no projects in `Partne…
b596137 
…rs Page`
@royallsilwallz
Left align Add/Edit Partners content
0dd68f6
@royallsilwallz
Adjust content spacing in Partners Page
6220850
@royallsilwallz
Fix mismatch value for language json file for Traditional Chinese
b02fafe 
- Related to hotosm#6456
@royallsilwallz
Changes in Partners Page Design
c3a0f16 
- Design improvements to Partners Statistics Page
- Fix UI misalignments
- Related to hotosm#6462
@royallsilwallz
Add Resources tab for other links in Partners Page
a6e5e5d
@royallsilwallz
Show Resources tab only when links are available
736cd5f
@royallsilwallz
Fix grey background color in switch component
04efae7
@royallsilwallz
Show only 2 primary hashtags in Partners card view
b68cdb7
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
af1b0a8 
for more information, see https://pre-commit.ci
@ramyaragupathy
Merge pull request hotosm#6490 from hotosm/fix/6462-partners-page-issues
9feaf0b 
Fixes for Partners Page UI
@dakotabenjamin
Add base url referrer to Nominatum api
059cc2a
@spwoodcock @dakotabenjamin
build: update cloudformation config to use pdm export instead of venv
2219239
@spwoodcock @dakotabenjamin
build: add --user flag to cloudformation pip install (avoid sys confl…
735f260 
…icts)
@dakotabenjamin
Set path to use root user local dir for python packages installed loc…
1c5f01a 
…ally
@dakotabenjamin
Merge pull request hotosm#6494 from hotosm/fix/pdm-develop-cherrypicked
c0e7193 
Fix/pdm develop cherrypicked
@dakotabenjamin
Merge pull request hotosm#6414 from hotosm/deploy-frontend-tests
4696687 
Frontend tests now has parallel workers & removed cluttered warnings
@prabinoid
Partners table atrributes lengths adjusted
dd0bd5d
@ramyaragupathy
Merge pull request hotosm#6498 from hotosm/fix/partners-attributes-le…
05877fb 
…ngth

Partners table atrributes lengths adjusted
@royallsilwallz
Fix activity section loading forever on secondary hashtag empty issue
c643248 
- Related to hotosm#6496
@royallsilwallz
Display partner name in the banner when logo not present
07f1a8b 
- Partners Statistics page
- Related to hotosm#6497
VinayakRugvedi and others added 24 commits September 6, 2024 19:12
@VinayakRugvedi @bshankar
feat: handle empty state of projects table
de9ec02
@bshankar
Fix: Cannot set omitMapResults when downloadAsCSV is true
c2e69e0 
Use df.drop instead of del with the pandas dataframe
@bshankar
Feat: Add organisation name to CSV
06b0d66
@bshankar
Fix: Rename CSV column total_area -> totalAreaSquareKilometers
900d43b
@bshankar
Chore: Add a comment to clarify total area computation
28ce653
@VinayakRugvedi @bshankar
Show tooltip content only if present
6fd975a
@VinayakRugvedi @bshankar
Add custom tooltip to progress, set omitMapResults to undefined for C…
5d0e16e 
…SV download, and remove table stripe
@VinayakRugvedi @bshankar
Handle empty data column, update basic styling of table, and add titl…
cc816d2 
…e attribute
@VinayakRugvedi @bshankar
refactor: use conditional operator, simplify expression
9f2bf02
@VinayakRugvedi @bshankar
fix: change medium to moderate, ensure tooltip is not masked by progr…
89054f1 
…ess bar
@VinayakRugvedi @bshankar
Set fill to currentColor on DownloadIcon
5f015ed
@VinayakRugvedi @bshankar
feat: replace anchor tag with thorough state and api integration for …
04952a7 
…download as CSV
@VinayakRugvedi @bshankar
Set minimum threshold width of progress bar
5a57a35
@VinayakRugvedi @bshankar
fix: remap query params to API spec while downloading as CSV
51c8c3b
@bshankar
Fix: percent_mapped and validated calculation in table view
2571921
@bshankar
Fix: CSV: Remove empty partnerNames column for non admins
28b61f3
@bshankar
Fix: CSV: Round total area in Sqkm to 3 decimal places
6bf7f2b
@dakotabenjamin
Merge pull request hotosm#6510 from hotosm/fix-docker-healthchecks
e26f3c7 
fix: db container healthchecks that was earlier broken
@spwoodcock
Merge pull request hotosm#6554 from spwoodcock/build/cleanup-dockerfiles
d639285 
Cleanup dockerfiles, remove duplicates, upgrade PDM
@dakotabenjamin
Temporarily disable automatic deployment of dev branch to servers for…
6ae8c1c 
… e2e testing
@pre-commit-ci
[pre-commit.ci] pre-commit autoupdate
0eef46a 
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.6.3 → v0.6.4](astral-sh/ruff-pre-commit@v0.6.3...v0.6.4)
@dakotabenjamin
Merge pull request hotosm#6559 from hotosm/develop-disable-ci
1126500 
Temporarily disable automatic deployment of dev branch to servers for e2e testing
@ramyaragupathy
Merge pull request hotosm#6504 from bshankar/feature/6500-project-sum…
249ae84 
…mary-table-view

Project summary table view
@spwoodcock
Merge pull request hotosm#6560 from hotosm/pre-commit-ci-update-config
525a972 
[pre-commit.ci] pre-commit autoupdate
@royallsilwallz royallsilwallz force-pushed the upgrade/iD-editor-2.30.2 branch from 681077b to 8390fd7 Compare September 17, 2024 07:31
@prabinoid prabinoid merged commit cae8fb4 into deployment/naxa Dec 18, 2024
7 checks passed
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
dependencies scope: backend scope: frontend scope: infrastructure scope: translations
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.