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

fix(noah): displaying hospital icon and label; fix display summary data; movable pin in map #384

Merged
merged 46 commits into from
Jul 29, 2024

Conversation

kennethliporada
Copy link
Collaborator

Description

  • fix bug about displaying critical facilities icon and label

Screenshot

image

pfgoting and others added 30 commits October 5, 2021 21:51
Added github actions workflow for auto deployment to both s3 and asti servers
Changed to staging. Created branch for staging.
# Summary 

- Fixes #186
- The `load` event isn't firing on NOAH Studio. As such, the `centerListener()` method isn't called which is responsible for the zooming in to the selected place. However, `style.load` do get called. The fix implemented here is to also use the event `style.load` to call the methods that were previously called by upon the firing of the `load` event but only listen to `style.load` once.
  - We need to further investigate **why** `load` doesn't work on Studio in iOS but works in KYH
- Confirming that only the NOAH Studio is affected by this issue.
 
# Demo

![Kapture 2021-10-05 at 22 49 45](https://user-images.githubusercontent.com/11599005/136047077-d0da7a85-9157-4d01-bae9-07a6039fd9c1.gif)
* Added if condition to run workflow only on successful PR merge. (#195)

* Checking proper automation workflow. Will revert this small change upon validation. (#196)
* Checking proper automation workflow. Will revert this small change upon validation.

* Changed trigger for deployment workflow. Revert small change to bibliography.
* Added AWS deployment workflow
@kennethliporada kennethliporada added the bug Something isn't working label Jul 11, 2024
@kennethliporada kennethliporada self-assigned this Jul 11, 2024
Copy link

netlify bot commented Jul 11, 2024

Deploy Preview for noah-ph ready!

Name Link
🔨 Latest commit 204b223
🔍 Latest deploy log https://app.netlify.com/sites/noah-ph/deploys/66a783c3bd137d00079e9d53
😎 Deploy Preview https://deploy-preview-384--noah-ph.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@bon-carpo bon-carpo changed the title fix(crit fac): Displaying hospital icon and label fix(crit fac): displaying hospital icon and label Jul 17, 2024
@bon-carpo
Copy link
Collaborator

CHANGES

  • add lat long details when pin is dragged in kyh and noah studio
  • change color of the pin into red in both kyh and noah studio
  • remove text center of latest data in summary table
  • adjust summary range data in iot

SCREENSHOTS

image
image
image

BEFORE
image

@bon-carpo bon-carpo changed the title fix(crit fac): displaying hospital icon and label fix(noah): displaying hospital icon and label and fix display summary data and movable pin in map Jul 29, 2024
@bon-carpo bon-carpo changed the title fix(noah): displaying hospital icon and label and fix display summary data and movable pin in map fix(noah): displaying hospital icon and label; fix display summary data; movable pin in map Jul 29, 2024
@bon-carpo bon-carpo merged commit 037b867 into main Jul 29, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Development

Successfully merging this pull request may close these issues.

4 participants