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.

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

chore(map): Refactor Sagas [EP-3709] #443

Open
wants to merge 27 commits into
base: develop
Choose a base branch
from

Conversation

ancashoria
Copy link
Contributor

All boxes should be checked before the PR can be accepted. Exceptions can be made, but they need to be argumented and discussed with the reviewer


STORY

Required

  • Added QA classes on main containers and all actionable items like Buttons, Inputs, etc
  • Translated all the user facing text (static & dynamic)
  • Ran Unit Tests and they pass successfuly
  • Added unit tests for the code in earth-shared
  • Ran the linter with yarn lint
  • No TypeScript warnings are introduced
  • I have made corresponding changes to the documentation

Not Needed

  • Drag here



BUGFIX

Required

  • Translated all the user facing text (static & dynamic)
  • Ran Unit Tests and they pass successfuly
  • Ran the linter with yarn lint
  • No TypeScript warnings are introduced
  • I have made corresponding changes to the documentation

Not Needed

  • Drag here



References

Include any links supporting this change such as a:

  • GitHub Issue/PR number addressed or fixed
  • StackOverflow post
  • Support forum thread
  • Related pull requests/issues from other repos

If there are no references, simply delete this section.

By submitting a PR to this repository, you agree to the terms within the Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.

ancashoria and others added 26 commits January 22, 2021 10:37
* feat(map) - update legend container max-height to be aligned with the search-box [EP-3645] (#442)

Signed-off-by: Vlad Tomsa <tomsavlad90@gmail.com>

* feat(profile) - add country and institution fields (#440)

Signed-off-by: Vlad Tomsa <tomsavlad90@gmail.com>

* Ep 3679 add institution and country to sign up form (#441)

* feat(auth) - add new fields to sign-up screen

Signed-off-by: Vlad Tomsa <tomsavlad90@gmail.com>

* fix(auth) - update sign up styling

Signed-off-by: Vlad Tomsa <tomsavlad90@gmail.com>

* feat(auth) - add new fields to sign-up screen

Signed-off-by: Vlad Tomsa <tomsavlad90@gmail.com>

* fix(auth) - update sign up styling

Signed-off-by: Vlad Tomsa <tomsavlad90@gmail.com>

* feat(auth) - use country list provided by the API

Signed-off-by: Vlad Tomsa <tomsavlad90@gmail.com>

* fix(auth) - remove shared country list

Signed-off-by: Vlad Tomsa <tomsavlad90@gmail.com>

* feat(auth) - update country select styling

Signed-off-by: Vlad Tomsa <tomsavlad90@gmail.com>

* fix(auth) - code refactor

Signed-off-by: Vlad Tomsa <tomsavlad90@gmail.com>

* feat(auth) - update dropdown styles

Signed-off-by: Vlad Tomsa <tomsavlad90@gmail.com>

* fix(map): weglot gatsby integration [EP-3710] (#445)

* feat(map): Set page background to dark [EP-3281]

* fix(marapp-admin): exclude edited layer from included layers dropdown [EP-3676]

* wip

* feat(map): support SWR on selected location [EP-3709]

Co-authored-by: Vlad Tomsa <tomsavlad90@gmail.com>
Co-authored-by: Horia Miron <horia.ancas@gmail.com>
Co-authored-by: Horia Iacos <iacos.horia@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants