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

Feature/6500 project summary table view #115

Merged
merged 175 commits into from
Aug 26, 2024
Merged

Feature/6500 project summary table view #115

merged 175 commits into from
Aug 26, 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
@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
@royallsilwallz
Refactor Partners card component
74519c5 
- Adjust height of the card
- Adjust spacing between card contents
- Add ellipsis to hashtags section
VinayakRugvedi and others added 25 commits August 22, 2024 12:28
@VinayakRugvedi @bshankar
Update messages for the explore projects table view
2f74d90
@VinayakRugvedi @bshankar
feat: create explore projects table view component
31e0d06
@VinayakRugvedi @bshankar
feat: define columns of table
d6f93eb
@VinayakRugvedi @bshankar
Add stylesheet for table to style resizer and striped rows
720d4da
@VinayakRugvedi @bshankar
Pass query param props to partner filter
15b5154
@VinayakRugvedi @bshankar
Update query specification and update more filters active state
ca17ae3
@VinayakRugvedi @bshankar
Update state handling and integrate with query params
0de659a
@VinayakRugvedi @bshankar
Handle overflow for explore projects table view
a4fdd70
@VinayakRugvedi @bshankar
feat: implement download CSV for project details
763d3d3
@VinayakRugvedi @bshankar
feat: update download CSV link
7d00b4a
@VinayakRugvedi @bshankar
feat: handle empty state of projects table
ecfad7c
@VinayakRugvedi @bshankar
Update behaviour of clear filters
9f031b0
@VinayakRugvedi @bshankar
Link name to details, mappers to contributors, and fix due date column
4452d9b
@VinayakRugvedi @bshankar
feat: add due soon, most mapped, and most validated options to sort by
effc98e
@VinayakRugvedi @bshankar
fix: dont set omitMapResults when downloadAsCSV is true
c5fbcf1
@VinayakRugvedi @bshankar
feat: handle empty state of projects table
ee43cfc
@bshankar
Fix: Cannot set omitMapResults when downloadAsCSV is true
ecc2bdb 
Use df.drop instead of del with the pandas dataframe
@pre-commit-ci @bshankar
[pre-commit.ci] auto fixes from pre-commit.com hooks
bc424ca 
for more information, see https://pre-commit.ci
@bshankar
Feat: Add organisation name to CSV
d9f1bf4
@bshankar
Fix: Rename CSV column total_area -> totalAreaSquareKilometers
45480ef
@bshankar
Chore: Add a comment to clarify total area computation
6fdad99
@VinayakRugvedi @bshankar
Show tooltip content only if present
3f4489b
@VinayakRugvedi @bshankar
Add custom tooltip to progress, set omitMapResults to undefined for C…
71e358d 
…SV download, and remove table stripe
@VinayakRugvedi @bshankar
Handle empty data column, update basic styling of table, and add titl…
cb58d1b 
…e attribute
@VinayakRugvedi @bshankar
refactor: use conditional operator, simplify expression
0d44e15
@royallsilwallz royallsilwallz merged commit 010c142 into deployment/naxa Aug 26, 2024
6 of 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
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@royallsilwallz @spwoodcock @mahesh-naxa @ramyaragupathy @dakotabenjamin @prabinoid @tsmock @bshankar @VinayakRugvedi