Skip to content

snapping spike#62

Closed
johnrees wants to merge 11 commits intomainfrom
jr/snapping
Closed

snapping spike#62
johnrees wants to merge 11 commits intomainfrom
jr/snapping

Conversation

@johnrees
Copy link
Copy Markdown
Contributor

@johnrees johnrees commented Sep 29, 2021

code is a mess, just seeing what's possible

snap.mp4

@netlify
Copy link
Copy Markdown

netlify Bot commented Sep 29, 2021

❌ Deploy Preview for oslmap failed.

🔨 Explore the source changes: 01f17e8

🔍 Inspect the deploy log: https://app.netlify.com/sites/oslmap/deploys/618b960b88f03b00078b4e23

Comment thread src/my-map.ts Outdated
})
);
});
});
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👀 juicy!! I hadn't really thought through how to access/construct a points layer yet from the basemap - this is smart!!

@jessicamcinchak
Copy link
Copy Markdown
Member

#55

@jessicamcinchak
Copy link
Copy Markdown
Member

@jessicamcinchak
Copy link
Copy Markdown
Member

Ok sorry I think my squashed commits are making Netlify complain about not-real issues! All our same commits are building fine over here: #75 (maybe because i squashed into your branch?)

Screenshot from 2021-11-10 11-07-14

Anyways closing this to avoid confusion, will open #75 for review !

@jessicamcinchak jessicamcinchak deleted the jr/snapping branch August 5, 2022 11:10
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.

2 participants