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

1393-add-icon-config-scrapers #1422

Merged
merged 3 commits into from
Sep 27, 2023
Merged

1393-add-icon-config-scrapers #1422

merged 3 commits into from
Sep 27, 2023

Conversation

mainawycliffe
Copy link
Contributor

Closes #1393

@netlify
Copy link

netlify bot commented Sep 21, 2023

Deploy Preview for goofy-euclid-75956c ready!

Name Link
🔨 Latest commit 60a79cf
🔍 Latest deploy log https://app.netlify.com/sites/goofy-euclid-75956c/deploys/651344949b783d00079c082c
😎 Deploy Preview https://deploy-preview-1422--goofy-euclid-75956c.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.

@netlify
Copy link

netlify bot commented Sep 21, 2023

Deploy Preview for flanksource-demo-stable ready!

Name Link
🔨 Latest commit 60a79cf
🔍 Latest deploy log https://app.netlify.com/sites/flanksource-demo-stable/deploys/65134494658e7f0008961bee
😎 Deploy Preview https://deploy-preview-1422--flanksource-demo-stable.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.

@netlify
Copy link

netlify bot commented Sep 21, 2023

Deploy Preview for clerk-saas-ui ready!

Name Link
🔨 Latest commit 60a79cf
🔍 Latest deploy log https://app.netlify.com/sites/clerk-saas-ui/deploys/65134494abba8f0009c2d1cb
😎 Deploy Preview https://deploy-preview-1422--clerk-saas-ui.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.

@moshloop
Copy link
Member

@mainawycliffe - I don't see the icons anywhere ?

@mainawycliffe
Copy link
Contributor Author

@moshloop they field? Should be next to the schedule field, inside the form section

@moshloop
Copy link
Member

@mainawycliffe config scrapers don't need an icon field, they need in icon the table in the same way connections have a connection icon in the table

@mainawycliffe mainawycliffe force-pushed the 1393-add-icon-config-scrapers branch from 0df44f7 to 5694c85 Compare September 25, 2023 12:21
@mainawycliffe
Copy link
Contributor Author

Ah, I see. Will make the changes.

@mainawycliffe mainawycliffe force-pushed the 1393-add-icon-config-scrapers branch from 5694c85 to 69f48d3 Compare September 25, 2023 12:25
@mainawycliffe
Copy link
Contributor Author

What will the icon be based on, the name?

@mainawycliffe mainawycliffe force-pushed the 1393-add-icon-config-scrapers branch from 69f48d3 to a377740 Compare September 25, 2023 12:35
@mainawycliffe mainawycliffe marked this pull request as draft September 25, 2023 12:35
@mainawycliffe mainawycliffe force-pushed the 1393-add-icon-config-scrapers branch from a377740 to d6c8089 Compare September 26, 2023 19:51
@mainawycliffe mainawycliffe force-pushed the 1393-add-icon-config-scrapers branch from d6c8089 to 60a79cf Compare September 26, 2023 20:52
@mainawycliffe mainawycliffe marked this pull request as ready for review September 26, 2023 20:52
@moshloop moshloop merged commit 78c8191 into main Sep 27, 2023
9 checks passed
@moshloop moshloop deleted the 1393-add-icon-config-scrapers branch September 27, 2023 06:57
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.

Add icon for config scrapers
2 participants