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

draft: wip(studio): display windy #247

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

draft: wip(studio): display windy #247

wants to merge 15 commits into from

Conversation

kennethliporada
Copy link
Collaborator

@kennethliporada kennethliporada commented Dec 21, 2021

Description

  • I created expand and show in sidebar

Screenshot

image

Error

  • I can't display wind in map
  • for exploration about this.
    image

Question

  • Paano ko ma call yung javascript ni Windy via s3? link here

  • Ito po kasi part na yan irread yung Windy na function name.
    image

  • ito po yung sa mismong repo ni sir prince na windy.js
    image

pfgoting and others added 15 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
@netlify
Copy link

netlify bot commented Dec 21, 2021

✔️ Deploy Preview for noah-ph ready!

🔨 Explore the source changes: b995ae7

🔍 Inspect the deploy log: https://app.netlify.com/sites/noah-ph/deploys/61c1864bbb6d860008383442

😎 Browse the preview: https://deploy-preview-247--noah-ph.netlify.app/

@kennethliporada kennethliporada changed the title draft: feat(studio): display windy draft: wip(studio): display windy Dec 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants