Skip to content

Conversation

@DennisPitallano
Copy link
Contributor

@DennisPitallano DennisPitallano commented Jan 11, 2025

issue #133

The social links highlighted in the screenshots have been removed from the website. You can preview and visually review the changes here: https://dg-bio.vercel.app/.

image

image

image

@DGBNOOB
Copy link
Member

DGBNOOB commented Jan 13, 2025

Thank you Dennis,

I wasn’t anticipating moving further than the header links, but this works. The Vercel app is very helpful in seeing the proposed changes, which helps me get a glimpse of what the changes will look like before they’re made on the master branch. Looking at the changes, the wiki tab sitting by itself looks out of place, and I think it should be moved to the right of downloads.
wiki to the right of downloads

This PR also addresses changes to the community cards. @ycagel, can you provide further direction and clarification? In the Draft DIP, it says to remove the entire modules for DigiByte Alliance and DGBAT and to relocate them to the DigiByte-Core/digibyte discussion forum. Should the community sub-headers for DigiByte Alliance and DGBAT be left on the website with links that direct viewers to the GitHub discussion? And who is creating the discussions in the DigiByte-Core/digibyte discussion forum?

One small change is needed in the DigiByte Developers Card: Remove "DigiByte Developers on Gitter" and replace it with "DigiByte Developers on GitHub Discussions" with a link to the DigiByte-Core/digibyte discussion forum.

In the Draft DIP, it says Community: Social Media Remove: Remove Facebook, LinkedIn, Instagram, YouTube, Reddit, Medium. Replace: List X and GitHub. It is not specified in the Draft DIP where to direct the website visitor on GitHub; however, it should be to DigiByte Developers on GitHub Discussions with a link to the DigiByte-Core/digibyte discussion forum.

@ycagel, do you think this should be included or left blank since it’s only two social media icons? If it’s left blank, then this card is complete, but if X and GitHub are to be added, then we need to include a Community: Social Media card. The description will also have to be modified and translated across the other language versions of the website: “Follow DigiByte across all primary social media platforms to keep up with the latest DigiByte news and share the posts with your friends to help us make our voices heard all over the world.”

Dennis, Community: Meet the Creator of DigiByte Remove: The entire module should be removed. Replace: Nothing. This card is also set to be removed. You didn’t include it, and that’s fine, but it will be added to a future issue to be removed. Just wanted to bring it up since it’s in this section, or we can come back to it later. Same for the Community: Social Media Disclaimer.

Page Footer: DigiByte Links Remove: Includes the removal of Blockchain 2035.

This is my initial assessment of the proposed changes; I think we need a little more clarification before moving forward with this PR.

Dennis thank you as always for volunteering and picking up this task to improve the community website.

@ycagel
Copy link
Member

ycagel commented Jan 16, 2025

Thank you for posting the questions @DGBNOOB. I think @gto90 would be a better person to ask and get feedback from.

Copy link
Member

@gto90 gto90 left a comment

Choose a reason for hiding this comment

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

@DennisPitallano ,

Thank you so much for volunteering your time to implement these changes. I think this is almost ready to go, except that we need to remove the block regarding the founder of the blockchain and the reference to the Blockchain 2035 book in the footer.

One final change request would be to add a link in the Community section of the page's Wiki block to the Wiki page here: https://github.com/DigiByte-Core/digibyte/wiki/Community

@DennisPitallano
Copy link
Contributor Author

Hi @DGBNOOB , @gto90

I’ve made the updates based on your feedback:

Let me know if you have any further suggestions!

@DGBNOOB
Copy link
Member

DGBNOOB commented Jan 22, 2025

@DennisPitallano thank you as always,
I think this is a great start, once these changes are reviewed and implemented, we can move onto the next round of changes. Once the majority or all of the changes are made, we can look at refining if needed. thank you again, the Vercel app made it easy to see the changes. Just one question, Jared recently updated where it says v7.17.3 to v8.22.0 in the header and in downloads but it still shows v7.17.3 in the Vercel app will that change v8.22.0 back to v7.17.3 when the PR is merged?
Starting review now.

@DennisPitallano
Copy link
Contributor Author

Hi Jose @DGBNOOB ,

I just synced the latest changes from the main repo to my fork, so Jared's updates should also be reflected on the Vercel app.

DGBNOOB
DGBNOOB previously approved these changes Jan 22, 2025
Copy link
Member

@DGBNOOB DGBNOOB left a comment

Choose a reason for hiding this comment

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

These are the notes I took to help me review the different commits and files:
7 commits 42538af, 3ec53e3, 8eecd23, 0674c24, ddf00c9, 97ffef9 and e237cc2.
-42538af:
removal dgballiance, dgbat, support
removal dgballiance dgbat
removal of social media card
removal of telegram footer
removal of social media footer
removal of dgballiance and dgbat from digibyte links
lang menu ticker removes pipeline and socials
-3ec53e3:
replaces gitter with GitHub discussions
-8eecd23:
removes meet the creator card
-0674c24:
removes blockchain 2035 link
-ddf00c9:
adds "https://github.com/DigiByte-Core/digibyte/wiki/Community" in the DigiByte Wiki card

-97ffef9
wiki link to header

-e237cc2, 38 files changed, +155 -5,058 lines
Pull Request (PR) 134, Addresses Issue 133 Remove header info from the DigiByte website.
The wiki tab is relocated to the right of downloads at the top, I think this pertains more to website visitors on a laptop. This PR also removes Community Cards: DigiByte Alliance, DigiByte Awareness Team, Social Media, Telegram groups, and replaces Gitter with DigiByte Developers on GitHub Discussions. There is a direct link from Community card “The DigiByte WIKI, by the community, for the community which goes to DigiByte-Core/digibyte/wiki/community”. The “Meet the creator” card is removed. This PR also removes footer info: Telegram channels, DigiByte links to DigiByte Alliance, DigiByte Awareness Team, Blockchain 2035, and social media links.
The first change relocates the Wiki tab to the right of downloads
The next set of changes removes dgballiance, dgbat, wiki, and support
The next set of changes remove dgballiance and dgbat information
The next change replaces Gitter to Github discussions
The next set of changes removes social media and telegram icons and links
The next change is in the Wiki community card where it adds a link to DigiByte-Core/digibyte/wiki/Community.
Removal of telegram footer icons/links
Removal of Social media icons and links
Removal of dgballiance, dgbbat, and 2035 from DigiByte links.
There were also some minor edits to the HTML, everything looks good.
Fantastic job Dennis.

Recommend approval cACK.

@ycagel ycagel requested a review from gto90 January 23, 2025 13:11
@gto90 gto90 added the enhancement New feature or request label Mar 3, 2025
@gto90
Copy link
Member

gto90 commented Mar 3, 2025

@DennisPitallano, this looks pretty great, and I appreciate your hard work. There are still some lingering community links concerning DigiID (apps that don't exist anymore) and mentions of community repositories and members that still need to be migrated to the Community section of the wiki. Before merging this, we must set up the Community wiki page, link this PR to the DIP, and move it to the next DIP stage.

I will do my best to set up that wiki page with all of the links to community socials, chat app channels, and contributor links this coming week. Once that is done, we can move the DIP to the next phase.

@gto90
Copy link
Member

gto90 commented Mar 5, 2025

Alright @DennisPitallano , we have migrated all of the community links to the DigiByte-Core/digibyte wiki here.

I think we need to remove the AntumID link as it's a dead link. We need to confirm that Coinomi still supports authenticating with DigiID, and if not, it also needs to be removed from the DigiID section. The same with Ownbit.

At the bottom of the page the Digibyte Mining needs to point to Mining. Additionally, we need to get that new wiki page populated with content which is not in scope for this PR.

@ycagel
Copy link
Member

ycagel commented Mar 6, 2025

Coinomi still supports Digi ID based on @DGBNOOB's finding.

@DGBNOOB
Copy link
Member

DGBNOOB commented Mar 6, 2025

Coinomi still supports Digi ID based on @DGBNOOB's finding.

I can confirm that Digi-ID is still supported on Coinomi. I read a question in a separate group asking if Digi-ID still works on the Coinomi wallet, I used it to login to DigiNexum.trade an hour ago.

I also just tested Digi-ID on Ownbit and it’s still supported.

@DennisPitallano
Copy link
Contributor Author

I’ve made the updates to remove the AntumID link and updated the mining link to the Wiki.
5606d1c
918245d

@ycagel ycagel requested a review from DGBNOOB March 7, 2025 01:52
Copy link
Member

@DGBNOOB DGBNOOB left a comment

Choose a reason for hiding this comment

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

The vercel.app made it easy to see the proposed changes. Three new commits were added since last viewed.

Merge branch DigiByte-Core:master into master 77c6fd3 +1 -1 lines changed

This change adds "-final" to https://github.com/DigiByte-Core/digibyte/releases/tag/v8.22.0-final, which was previously missing and directed users to the repository instead of Releases.

Remove the AntumID link from Digi-ID apps: Commit 5606d1c +0 -407 lines changed 37 files changed. af-zh language files to include en-us, pt-br, and an index.html file.

Update mining wiki link: Commit 918245d +37 -37 lines changed 37 files changed. Added to af-zh language files to include en-us, pt-br, and an index.html file.

also reviewed changes since last viewed
38/38 files viewed. +192-5,502 lines changed.

@DennisPitallano thank you again, awesome work.
@gto90 great job with the community wiki page.

I recommend approval of PR 134 cACK.

Copy link
Member

@gto90 gto90 left a comment

Choose a reason for hiding this comment

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

ACK

Copy link
Contributor

@JaredTate JaredTate left a comment

Choose a reason for hiding this comment

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

cACK. Thank you all for taking the time to do this!

@gto90 gto90 merged commit 3e5f516 into DigiByte-Core:master Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants