8000 New Founder View with Progress Bar by yashop7 · Pull Request #393 · block-core/angor · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

New Founder View with Progress Bar #393

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 4 commits into from
Jun 4, 2025

Conversation

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

Making the Page simpler & easier to navigate, similar to Angor Hub

  • Introduced the Progress bar
  • Making Whole Card Clickable
  • Removed other Buttons

Before:
Screenshot 2025-05-21 at 2 08 30 AM

After:
Screenshot 2025-05-21 at 2 08 45 AM

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

@dangershony Can you plz review this

@dangershony
Copy link
Member

What about the buttons? they disappeared

@yashop7
Copy link
Contributor Author
yashop7 commented Jun 2, 2025

Hi Dan, this issue was mentioned in the planka, https://planka.angor.io/cards/1497764001369556685

@dangershony
Copy link
Member

Hi Dan, this issue was mentioned in the planka, https://planka.angor.io/cards/1497764001369556685

ah right I remember now ok good job

@dangershony
Copy link
Member
dangershony commented Jun 2, 2025

Can you add a small icon for projects that have messages?
we already have that info why not show it, it is the InvestmentRequests flag

image

@dangershony
Copy link
Member

I wonder if in the page view.razor we should also call the method LookupSignaturesDirectMessagesForPubKeyAsync and put a small indicator on the text box that there are pending messages
on this box.

perhaps do it in a separate pr?
image

@yashop7
Copy link
Contributor Author
yashop7 commented Jun 2, 2025

OK, Dan, will make the PR for this issue

@yashop7
Copy link
Contributor Author
yashop7 commented Jun 2, 2025
Screenshot 2025-06-02 at 8 41 39 PM

SVG will also redirect to /signatures/projectIdentifier (if there are requests and SVG will also show at that time) + TBTC fixed

@dangershony
Copy link
Member
Screenshot 2025-06-02 at 8 41 39 PM SVG will also redirect to /signatures/projectIdentifier _(if there are requests and SVG will also show at that time)_ + TBTC fixed

Brilliant that looks good

<strong>
@Money.Satoshis(FounderProject.ProjectInfo.TargetAmount).ToUnit(MoneyUnit.BTC)
</strong>
TBTC
Copy link
Member

Choose a reason for hiding this comment

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

no there is a method that will tell you the netwrok type use @network.CoinTicker for that

Copy link
Member

Choose a reason for hiding this comment

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

otherwise when we go to mainnet it will show TBTC that doesnt make sense

<strong>
@Money.Satoshis(FounderProject.ProjectInfo.TargetAmount).ToUnit(MoneyUnit.BTC)
</strong>
TBTC
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
TBTC
@network.CoinTicker

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done Dan 👍

@dangershony dangershony merged commit d70b892 into block-core:main Jun 4, 2025
3 checks passed
@yashop7 yashop7 deleted the fix-founder-view branch June 6, 2025 09:08
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.

2 participants
0