8000 Improve create page by miladsoft · Pull Request #383 · block-core/angor · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Improve create page #383

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 11 commits into from
May 16, 2025
Merged

Improve create page #383

merged 11 commits into from
May 16, 2025

Conversation

miladsoft
Copy link
Member

No description provided.

miladsoft added 4 commits May 5, 2025 17:45
…nd penalty days fields with improved labels and descriptions
…ate, and calculate expiry date automatically
…prove accessibility, and set automatic date handling for project stages
@miladsoft miladsoft marked this pull request as draft May 15, 2025 12:40
public string NostrPubKey { get; set; }
public DateTime StartDate { get; set; }
public DateTime EndDate { get; set; }
Copy link
Member

Choose a reason for hiding this comment

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

This goes in the nostr event, we must be careful what params go in here

Copy link
Member Author

Choose a reason for hiding this comment

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

We need to specify the end date of the fundraising. In previous discussions I asked if the first stage date meant the end date, which you said is not always the case.

Copy link
Member
@dangershony dangershony May 16, 2025

Choose a reason for hiding this comment

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

yes yes agree, but how does it work for existing projects?
WiIl it not break them?

Copy link
Member

Choose a reason for hiding this comment

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

do we need to use the version upgrade I wonder?

Copy link
Member Author

Choose a reason for hiding this comment

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

There is no problem for others. New projects specify the end date of the investment. In old projects, we define the start date of the stages as the end of the investment.

dangershony
dangershony previously approved these changes May 15, 2025
Copy link
Member
@dangershony dangershony left a comment

Choose a reason for hiding this comment

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

I have not tested this, I will test tomorrow

@miladsoft miladsoft marked this pull request as ready for review May 16, 2025 10:12
@dangershony dangershony merged commit b167f11 into main May 16, 2025
3 checks passed
@dangershony dangershony deleted the create-page branch May 16, 2025 15:43
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