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

feat: social proof #686

Merged
merged 12 commits into from
Apr 28, 2023
Merged

feat: social proof #686

merged 12 commits into from
Apr 28, 2023

Conversation

jjagielka
Copy link
Contributor

@jjagielka jjagielka commented Apr 22, 2023

πŸ“‘ Description

Social proof components.

  • need to verify icons
  • need to implement to get hits dynamically
  • need to update copy writing
  • need to pull Stars and Download from Flowbite-Svelte
  • Not Completed
  • Completed

βœ… Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • I have checked the page with https://validator.unl.edu/
  • All the tests have passed
  • My pull request is based on the latest commit (not the npm version).

β„Ή Additional Information

@stackblitz
Copy link

stackblitz bot commented Apr 22, 2023

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@vercel
Copy link

vercel bot commented Apr 22, 2023

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

Name Status Preview Comments Updated (UTC)
flowbite-svelte βœ… Ready (Inspect) Visit Preview πŸ’¬ Add feedback Apr 28, 2023 9:48pm
flowbite-svelte-update βœ… Ready (Inspect) Visit Preview πŸ’¬ Add feedback Apr 28, 2023 9:48pm

@shinokada
Copy link
Collaborator

shinokada commented Apr 22, 2023

  • @zoltanszogyenyi I'm not sure about the copy because it is about Flowbite not about Flowbite-Svelte. Isn't it confusing to users?
  • Stars and Downloads are not from Flowbite-Svelte and I think it can be misleading users.

@shinokada
Copy link
Collaborator

@zoltanszogyenyi We need to update Get Started copy as well.

image

@jjagielka
Copy link
Contributor Author

Do you know how the get stats for Figma and Discord? I was search through the net, but there was nothing publicly open i.e. you need to set up the API client key. GitHub and NPM: no login required.

@shinokada
Copy link
Collaborator

shinokada commented Apr 26, 2023

It seems you need the server id and discord bot auth token.
@zoltanszogyenyi may know about it.

Discord Developer Portal

const serverId = 'YOUR_SERVER_ID';
const authToken = 'YOUR_BOT_AUTH_TOKEN';

fetch(`https://discord.com/api/guilds/${serverId}`, {
  headers: {
    'Authorization': `Bot ${authToken}`
  }
})
.then(response => response.json())
.then(data => {
  const onlineMembers = data.approximate_presence_count;
  console.log(`Number of members online: ${onlineMembers}`);
})
.catch(error => {
  console.error('Error fetching server information:', error);
});

@jjagielka
Copy link
Contributor Author

Yes, that's the way I found. I was thinking that you would prefer to register the bot under Themesberg name and just share a token with me.

@shinokada
Copy link
Collaborator

I am in the same position as you on the Flowbite Discord channel. @zoltanszogyenyi should have access to a setting for the token.

@zoltanszogyenyi
Copy link
Member

Hey @jjagielka @shinokada,

I pulled the data from Discord myself in a commit.

I think that for Figma we can keep it hardcoded for now - the only viable option is scraping the page.

Cheers,
Zoltan

@shinokada
Copy link
Collaborator

Fixed merge conflicts.

@jjagielka jjagielka deleted the social_proof branch April 29, 2023 05:15
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.

None yet

3 participants