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

Nice to have enhancements #167

Open
buhtignew opened this issue Aug 1, 2024 · 4 comments
Open

Nice to have enhancements #167

buhtignew opened this issue Aug 1, 2024 · 4 comments
Labels
enhancement New feature or request

Comments

@buhtignew
Copy link
Collaborator

buhtignew commented Aug 1, 2024

Instead of creating a thread for each small command's improvements that pops up while using them I've thought we can have a general thread like that of the Typos just putting everything which may seem to be useful here.
If you think that this thread is not helpful it this stage let's just close it.
_
I'm not 100% sure whether we've already discussed this or not but what if we suppress the -a flag in the address set command so as the user could just put the label or the address indifferently in the first position? Otherwise the user needs to know the label of the address, which sometimes can require a bit of additional research or he should remember that the -a flag before the address is needed. That's of course not a big deal, but maybe it would improve a little bit the usability.

@buhtignew buhtignew added the enhancement New feature or request label Aug 1, 2024
@d5000
Copy link

d5000 commented Aug 19, 2024

I'm still not working on this but adding the following suggestion from the help thread, which is not help-related:

IDK whether it would make sense for the average user, but during my testing I've realized that it would be useful to have the address list displaying the block on which the data is displayed.

I'm considering it.

@buhtignew
Copy link
Collaborator Author

buhtignew commented Aug 24, 2024

I think it would be nice to have a command or a flag that would enable to search a deck by parameters.
For instance if I launch token show ea7772ba8b5b297a35a50fac94431db5fc73731dfa7314dd53394d919ca4af58 -i I can see the complete info about the deck ea7772ba8b5b297a35a50fac94431db5fc73731dfa7314dd53394d919ca4af58, but what if I'd like to find a deck that has that at_address or that startblock (one of parameters included in the above command's output) without knowing the deck's id in advance?
_

As mentioned here I think it would be suitable to add a compatibility check into the pobtoken claim command in order to prevent people spending their fees on claiming tokens for the decks with the startblock that occurs after the burning transaction was created.
The same for the attoken claim command.
_

As mentioned here I think it would be nice to have not only the courtesy message for the PoD tokens used accidentally with transaction list DECK -b usage mode (for instance transaction list DTNewSpecv2 -b) but also for the ATtokens (or any other token that is not PoB) used by chance with the same usage mode (for instance transaction list ATTokenNotPoB -b).
If we opt for this solution the message for all the cases other than with the PoB token (i.e. transaction list PoD_token -b, transaction list ATToken -b, transaction list ANY_NOT_POB_TOKEN -b) should be changed from Deck xxxxxxxx is not an AT or PoB token deck. into Deck xxxxxxxx is not a PoB token deck.

@buhtignew buhtignew changed the title Nice to have command's enhancements Nice to have enhancements Oct 24, 2024
@d5000
Copy link

d5000 commented Oct 28, 2024

I think currently it's good as it is and thus I'm closing:

  • message when AT tokens are used is "Gateway address mismatch. Probably you are using a command for PoB tokens for an AT token. Please use the command/flag for AT tokens instead."
  • message when dPoD or other non-AT tokens is "Error: Deck DECK is not an AT or PoB token deck."

Both are different checks, thus it makes sense that the messages are different, and the first error can potentially also be thrown in other situations, thus the "probably".

@d5000 d5000 closed this as completed Oct 28, 2024
@buhtignew
Copy link
Collaborator Author

buhtignew commented Oct 29, 2024

I think your reply above refers to #209, which I'm closing, but this thread has other matters we are keeping there just in case we'll decide to implement them in the future.
I'm reopening.

@buhtignew buhtignew reopened this Oct 29, 2024
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

No branches or pull requests

2 participants