Skip to content

docs: update README to include all components#181

Merged
jessicamcinchak merged 2 commits intomainfrom
jess/update-README
Aug 16, 2022
Merged

docs: update README to include all components#181
jessicamcinchak merged 2 commits intomainfrom
jess/update-README

Conversation

@jessicamcinchak
Copy link
Copy Markdown
Member

No description provided.

@netlify
Copy link
Copy Markdown

netlify Bot commented Aug 16, 2022

Deploy Preview for oslmap ready!

Name Link
🔨 Latest commit 8620968
🔍 Latest deploy log https://app.netlify.com/sites/oslmap/deploys/62fb75c9d8ceb900091e79f5
😎 Deploy Preview https://deploy-preview-181--oslmap.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 settings.

Comment thread .npmrc
@@ -0,0 +1 @@
auto-install-peers=true
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

fixes pnpm warning
Screenshot from 2022-08-16 11-12-21

Comment thread README.md
@@ -1,16 +1,32 @@
# Map
# Place components
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Testing out a different name for this thing! How does it feel? See #22

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yeah I picked up on this change in the previous Pitsby PR - it works for me 👍 Other options would be location, gis, geospatial, spatial.

Let's go with Place!

Comment thread README.md
An [OpenLayers](https://openlayers.org/)-powered [Web Component](https://developer.mozilla.org/en-US/docs/Web/Web_Components) map for tasks related to planning permission in the UK.
A library of [Web Components](https://developer.mozilla.org/en-US/docs/Web/Web_Components) for tasks related to addresses and planning permission in the UK built with [Lit](https://lit.dev/), [Vite](https://vitejs.dev/), and [Ordnance Survey APIs](https://developer.ordnancesurvey.co.uk/).

***Web map***
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Again, related to #22, maybe in another PR I'll follow through with a proper rename from my-map to web-map - any other ideas??

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'd support that - makes a lot more sense than my-map.

Copy link
Copy Markdown
Contributor

@DafyddLlyr DafyddLlyr left a comment

Choose a reason for hiding this comment

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

Great stuff! 👍

Comment thread README.md
@@ -1,16 +1,32 @@
# Map
# Place components
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yeah I picked up on this change in the previous Pitsby PR - it works for me 👍 Other options would be location, gis, geospatial, spatial.

Let's go with Place!

Comment thread README.md
An [OpenLayers](https://openlayers.org/)-powered [Web Component](https://developer.mozilla.org/en-US/docs/Web/Web_Components) map for tasks related to planning permission in the UK.
A library of [Web Components](https://developer.mozilla.org/en-US/docs/Web/Web_Components) for tasks related to addresses and planning permission in the UK built with [Lit](https://lit.dev/), [Vite](https://vitejs.dev/), and [Ordnance Survey APIs](https://developer.ordnancesurvey.co.uk/).

***Web map***
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'd support that - makes a lot more sense than my-map.

Comment thread README.md
- Interactive web component docs [oslmap.netlify.app](https://oslmap.netlify.app)
- [CodeSandbox](https://codesandbox.io/s/confident-benz-rr0s9?file=/index.html) (note: update the CDN script with a version number for new features)

Find these components in the wild, including what we're learning through public beta user-testing, at [https://www.ripa.digital/](https://www.ripa.digital/).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is great to include.

@jessicamcinchak jessicamcinchak merged commit e555235 into main Aug 16, 2022
@jessicamcinchak jessicamcinchak deleted the jess/update-README branch August 16, 2022 10:48
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