Skip to content

Feature/dashboard maintenance refactor [IGNORE]#1583

Closed
mohitb35 wants to merge 40 commits into
feature/manage-payoutsfrom
feature/dashboard-maintenance-refactor
Closed

Feature/dashboard maintenance refactor [IGNORE]#1583
mohitb35 wants to merge 40 commits into
feature/manage-payoutsfrom
feature/dashboard-maintenance-refactor

Conversation

@mohitb35
Copy link
Copy Markdown
Member

@mohitb35 mohitb35 commented Oct 6, 2022

Refactors recently introduced features in the dashboard to remove code duplication / create reusable components / clean up code where possible. These features include

  1. Bulk Codes
  2. Create Donation Link
  3. My PlanetCash
  4. Manage Payouts

/// IGNORE BELOW THIS LINE
Fixes #

Check this list and then dismiss it:

  • Localization
    • Have you added translation resources?
    • Have you localized numbers, dates and other stuff?
  • Validation and error management
    • Is the user input validated?
    • Is the API response valided and are error responses properly handled?
    • Are errors caught if necessary?
  • Coding style
    • Have you used a specific type for declaring the properties and parameters instead of any?
    • Have you defined interfaces for new components?
    • Have you added a source file not using TypeScript?
    • Have you checked if your newly introduced modules or functions are not already existing?
    • Is your React.useEffect using the right dependencies?
    • Check your code if it matches the existing code formatting.

Changes in this pull request:

@

sunilsabatp and others added 30 commits August 9, 2022 14:36
@sunilsabatp
package-lock.json
472aaca
@sunilsabatp
change the redeem endpoint in profile
520bc19
@sunilsabatp
fix the error message for redeem code
8a43ba0
@sunilsabatp
fix codefactor error
fa20e79
@sunilsabatp
removed unused entities
51f1942
@sunilsabatp
fix client-side exception issue
0aad0ae
@sunilsabatp
fix the button and validation for redeem modal
b11c602
@sunilsabatp
changed name of validate button
f437e97
@sunilsabatp
pull the latest changes from remote
9719b4e
@sunilsabatp
fix redeem modal
50140c4
@sunilsabatp
added button for successful redeem modal
78687a8
@sunilsabatp
Merge branch 'develop' of https://github.com/Plant-for-the-Planet-org…
893b853 
…/planet-webapp into develop
@sunilsabatp
merged develop
24b3e99
@sunilsabatp
fix codefactor suggested error
7684044
@sunilsabatp
defined type for data
9a1a8e3
@sunilsabatp
defined type for userProfile
bb2ecad
@sunilsabatp
fix the redeem code through route
da4c1c6
@sunilsabatp
removed loaduser from redeemmodal
5afb945
@prachigarg19
center map
8df3f0f
@prachigarg19
replace dependency and condition
a230235
@prachigarg19
Revert "replace dependency and condition"
6deb008 
This reverts commit a230235.
@prachigarg19
change dependency and condition to center map
cf37c04
@prachigarg19
improve condition
42d2146
@prachigarg19
preserve coordinates when we return from single project page
28e298a
@sunilsabatp
clean up code
a1d5342
@sunilsabatp
fixed the structure of redeem modal
81bfe18
@mohitb35
refactor: minor cleanup
8ecddb1 
- remove unused state variables and console logs
- remove extra line spacing
- add return type for functional component
@sunilsabatp
fix structure of modal
d15a3e9
@sunilsabatp
cleaned up the repetitive code
a2a8aac
@mohitb35
fix: remove footer app links
791f913
mohitb35 and others added 9 commits September 29, 2022 16:05
@mohitb35
refactor: remove unused dependencies/fix TS errors
982b995
@sagararyal
Merge pull request #1576 from Plant-for-the-Planet-org/feature/remove…
64483d0 
…-footer-app-links

fix: remove footer app links
@mohitb35
feat: always show signup form for logged in user
a873daf 
- temporary fix. These changes will be reverted.
@mohitb35
feat: show space after end of form
ff1eb98 
- wider screens only (>480px)
- add a container div around form to allow fixed background
- update styling to mobile first styles
@mohitb35
fix: make form fields responsive
05c3d0b
@mohitb35
Revert "feat: always show signup form for logged in user"
bc6d6f6 
This reverts commit a873daf.
@mariahosfeld
Merge pull request #1554 from Plant-for-the-Planet-org/feature/redeem…
b767573 
…_endpoint

Feature/redeem endpoint
@mariahosfeld
Merge pull request #1570 from Plant-for-the-Planet-org/feature/center…
8ef4028 
…-map

Center map
@mariahosfeld
Merge pull request #1578 from Plant-for-the-Planet-org/feature/comple…
8d754fa 
…te-signup-responsiveness

Fix responsiveness issues for Complete Signup Form
@vercel
Copy link
Copy Markdown

vercel Bot commented Oct 6, 2022 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
planet-webapp ✅ Ready (Inspect) Visit Preview Oct 6, 2022 at 5:15AM (UTC)

@mohitb35 mohitb35 changed the base branch from develop to feature/manage-payouts October 6, 2022 05:13
@mohitb35 mohitb35 closed this Oct 6, 2022
@mohitb35 mohitb35 deleted the feature/dashboard-maintenance-refactor branch October 6, 2022 05:14
@mohitb35 mohitb35 changed the title Feature/dashboard maintenance refactor Feature/dashboard maintenance refactor [IGNORE] Oct 6, 2022
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

PR: unreviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@mohitb35 @sunilsabatp @prachigarg19 @sagararyal @mariahosfeld