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

Navigation and other interactive buttons now freeze when generating scrambles #632

Merged
merged 3 commits into from
Nov 14, 2020

Conversation

shzcuber
Copy link
Contributor

@shzcuber shzcuber commented Nov 9, 2020

No description provided.

@@ -19,6 +19,7 @@ const mapStateToProps = (store) => ({
scramblingProgressTarget: store.scramblingProgressTarget,
scramblingProgressCurrent: store.scramblingProgressCurrent,
fileZipBlob: store.fileZipBlob,
generatingScrambles: store.generatingScrambles,
Copy link
Member

Choose a reason for hiding this comment

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

Travis is complaining that this is duplicated. Could you please fix it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi, I made a new commit and pushed it onto this fork. Did it work? thanks!

@shzcuber
Copy link
Contributor Author

shzcuber commented Nov 9, 2020

I decided to undo all my unnecessary commits to make everything more clear. Hopefully this works now! Sorry for the confusion - I'm new to this.

@campos20
Copy link
Member

This is looking great! I have just one more request for you

Use 2014PRID01 and navigate to SCA Road Trip - Sydney 2021

This competition has FMC. Then, you can open the translation and then start generating scrambles. Input boxes are not disabled. Could you please disable it as well?

I think it's this one, if I'm not mistaken

@campos20 campos20 merged commit af0b9c1 into thewca:master Nov 14, 2020
@campos20
Copy link
Member

Thanks for taking care of this!

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.

None yet

2 participants