Skip to content

Conversation

@JonathanDHolden
Copy link
Contributor

added example link to Directory Page (link suggested by Ru to me on Slack) Also we need example links to Directory Venue and Directory Organisation (but I'm not sure which links to use)

What does this change?

How to test

How can we measure success?

Have we considered potential risks?

Images

Accessibility

added example link to Directory Page (link suggested by Ru to me on Slack)
Also we need example links to Directory Venue and Directory Organisation (but I'm not sure which links to use)
@netlify
Copy link

netlify bot commented Nov 9, 2024

Deploy Preview for inspiring-euclid-d918c8 ready!

Name Link
🔨 Latest commit d400be5
🔍 Latest deploy log https://app.netlify.com/sites/inspiring-euclid-d918c8/deploys/672f3bc24714290008a3b8b6
😎 Deploy Preview https://deploy-preview-291--inspiring-euclid-d918c8.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.

@andybroomfield
Copy link
Contributor

Thanks @JonathanDHolden looks like there are some markdown issues with the table.
https://deploy-preview-291--inspiring-euclid-d918c8.netlify.app/content/features/directories.html#concepts

Screenshot 2025-01-28 at 12 42 42 pm

Ask @willguv what needs linking here (example directories).

@finnlewis
Copy link
Member

@JonathanDHolden might you be able to review this one again?

@JonathanDHolden
Copy link
Contributor Author

From what I can see in the convo above, @willguv was going to be asked to what links are needed here

@willguv
Copy link
Member

willguv commented Feb 27, 2025

Do you need an example of a directory page (no geo) or a directory venue (with geo)?

@JonathanDHolden

@finnlewis
Copy link
Member

OK - passing to @willguv for a review and update if needed.

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.

4 participants