8000 Added Investor Link to angor indexer project page by yashop7 · Pull Request #370 · block-core/angor · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Added Investor Link to angor indexer project page #370

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

Merged
merged 3 commits into from
May 13, 2025

Conversation

yashop7
Copy link
Contributor
@yashop7 yashop7 commented May 11, 2025

added link to angor indexer project page to show list of investors in new tab
added noopener noreferrer for privacy

Before
Screenshot 2025-05-12 at 2 00 20 AM

After
Screenshot 2025-05-12 at 1 59 04 AM

@yashop7
Copy link
Contributor Author
yashop7 commented May 11, 2025

@miladsoft , the button i have added is 👍 or 👎

@@ -298,17 +298,22 @@
</div>

<div class="col-md-4">
<a href="https://fulcrum.angor.online/angor/projects/@project.ProjectInfo.ProjectIdentifier" target="_blank" rel="noopener noreferrer">
Copy link
Member

Choose a reason for hiding this comment

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

You should use a primary indexer, not a hardcoded address.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, I got the problem, does this Fix solve it?

</div>
<div class="stat-icon">
<Icon IconName="users" Height="32" Width="32" />
<a href="@($"{_NetworkService.GetPrimaryIndexer().Url}/angor/projects/{project.ProjectInfo.ProjectIdentifier}")" target="_blank" rel="noopener noreferrer">
Copy link
Member

Choose a reason for hiding this comment

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

it is not the indexer you want it is the explorer _NetworkService.GetPrimaryExplorer()
And please build the link itself in a method in the c# code

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Dan made the changes

@dangershony dangershony merged commit 31837d4 into block-core:main May 13, 2025
3 checks passed
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.

3 participants
0