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

production summary remove sheet doesnt work under a folder #79

Closed
apia46 opened this issue Mar 13, 2024 · 2 comments
Closed

production summary remove sheet doesnt work under a folder #79

apia46 opened this issue Mar 13, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@apia46
Copy link

apia46 commented Mar 13, 2024

v6.1.0, pyanodons ae + text plates
if i make a folder in a summary sheet and put a production sheet in it, i cant remove the production sheet with the remove button
https://github.com/have-fun-was-taken/yafc-ce/assets/96461146/993cf441-e0d3-4315-b382-d564f22ba65d

@apia46 apia46 added the bug Something isn't working label Mar 13, 2024
@veger
Copy link
Collaborator

veger commented Mar 14, 2024

The production summary is a partial feature implemented by the original YAFC author. So you are right, it does not have a feature complete implementation...

As we did not implement it, we also do not have a fully vision of this feature and we do not know what is lacking and what is not... It would have been nice to place some warning that this is an incomplete/alpha feature so users are aware that they should use this on their own risk.? (@have-fun-was-taken)

Additionally, I have been working on another production summary view myself in parallel... It has been merged and will be part of the next (0.6.2) release: #38

It uses a completely different vision than the production summary, so I guess we need to see how/if they can/should live next to each other. Or whether we need to merge them, or remove one of them.

So for now I doubt that we'll be adding missing features to the production summary until we have some clear idea what to do

@DaleStan
Copy link
Collaborator

I believe I fixed this as part of #231, released in 0.9.0. If you're still seeing it, feel free to reopen this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants