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

Create a special case view for Dcur2... #105

Closed
raedah opened this issue Aug 29, 2017 · 6 comments
Closed

Create a special case view for Dcur2... #105

raedah opened this issue Aug 29, 2017 · 6 comments
Milestone

Comments

@raedah
Copy link

raedah commented Aug 29, 2017

Dev subsidy address is Dcur2mcGjmENx4DhNqDctW5wJCVyT3Qeqkx
It accrues a higher balance with every block that is mined. Because of the large number of transactions, this page deserves its own template with a custom layout, and some of the data for this page should be pre-cached, such as the balance.

@chappjc chappjc added this to the 0.7.x milestone Aug 29, 2017
@chappjc
Copy link
Member

chappjc commented Aug 29, 2017

Almost a bug, and should be addressed in a patch version rather than 0.8.

@raedah
Copy link
Author

raedah commented Sep 3, 2017

So for cases with > 2000, I was thinking we could do something like mark those as special cases to be pre-cached. Could trigger on first load, and update during load as well. Having to display "loading..." just the first time it is hit is reasonable, but dont want to recalc every time the address is hit. And if the logic is done well, could easily add in new data to existing totals.

@chappjc chappjc modified the milestones: 0.8, 0.7.x Sep 6, 2017
@chappjc chappjc self-assigned this Sep 6, 2017
@chappjc
Copy link
Member

chappjc commented Sep 6, 2017

Ah, we can defer to 0.8.x since 2000 tx with Dcur2 is less than 1 MB.

@chappjc chappjc modified the milestones: tbd, 0.8 Sep 6, 2017
@chappjc
Copy link
Member

chappjc commented Nov 6, 2017

PostgreSQL handles this smoothly, and I've implemented caching of the summary information in PR #209.

However, I agree that this address could even use it's own page as it has special significance.

@chappjc chappjc removed their assignment Aug 29, 2018
@ukane-philemon
Copy link
Collaborator

@chappjc, Is this still required?

@chappjc
Copy link
Member

chappjc commented Dec 24, 2022

It's resolved. The link /treasury took the place of the old /address/Dcur2.... Plus the old address page gets show as "Legacy Decred Treasury". I think that this issue is resolved when considered with the address caching.

@chappjc chappjc closed this as completed Dec 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants