Skip to content

Refactor ajax scripts with flux with reactjs #2342

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.

Sign up for GitHub

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

Jump to bottom
Closed
jeff1evesque opened this issue Dec 25, 2015 · 2 comments
Closed

Refactor ajax scripts with flux with reactjs #2342

jeff1evesque opened this issue Dec 25, 2015 · 2 comments
Labels
build invalid new feature
Milestone
0.7

Comments

@jeff1evesque
Copy link
Owner

We will refactor our ajax scripts with flux, in conjunction with reactjs.

Note: currently we have our ajax scripts being referenced within the <head> element, then referenced within the global space, within our reactjs script, which is referenced within the <body>, of the same index.html.
@jeff1evesque jeff1evesque added this to the 0.3 milestone Dec 25, 2015
@jeff1evesque jeff1evesque changed the title Create and implement Centos 7.x vagrant base box Refactor ajax scripts with flux Dec 26, 2015
@jeff1evesque jeff1evesque changed the title Refactor ajax scripts with flux Refactor ajax scripts with flux, and integrate with reactjs Dec 26, 2015
@jeff1evesque jeff1evesque modified the milestones: 0.6, 0.3 Feb 17, 2016
@jeff1evesque
Copy link
Owner Author

Whether we choose to implement flux, or redux, we will need to refactor (modify, or eliminate) our this.state.mounted implementation, used in several of our ajax equivalent (fetch) scripts.

@jeff1evesque
Copy link
Owner Author

We've already implemented redux, as indicated by #2854. Additionally, we currently do not have this.state.mounted in our codebase:

jeff1evesque@XXXXXXX YYYYYYY ~/web-projects/machine-learning (feature-3058)
$ grep -r 'this.state.mounted' src/jsx
jeff1evesque@XXXXXXX YYYYYYY ~/web-projects/machine-learning (feature-3058)
$

Therefore, we can safely mark this issue as invalid, since it is no longer needed.

@jeff1evesque jeff1evesque changed the title Refactor ajax scripts with flux, and integrate with reactjs Refactor ajax scripts with flux with reactjs Nov 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
build invalid new feature
Projects
None yet
Milestone
0.7
Development

No branches or pull requests
1 participant
@jeff1evesque