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

Fixed creation of double instances in memory graph activity, this fixes #627 #632

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

htg2704
Copy link

@htg2704 htg2704 commented Feb 17, 2020

Fixes #[Add issue number here. Note: This will automatically closes the issue. If you do not solve the issue entirely, please change the message to e.g. "First steps for issues #IssueNumber]

Changes: [Add here what changes were made in this issue and if possible provide links.]

Screenshot/s for the changes: [Add screenshot/s of the layout where you made changes or a *.gif containing a demonstration. Use <img src="paste-url-here" width=200/> to add images]

Checklist: [Please tick following check boxes with [x] if the respective task is completed]

  • I have used resources from strings.xml, dimens.xml and colors.xml without hard-coding them
  • No modifications done at the end of resource files strings.xml, dimens.xml or colors.xml
  • I have reformatted code in every file included in this PR [CTRL+ALT+L]
  • My code does not contain any extra lines or extra spaces
  • I have requested reviews from other members

APK for testing: [Compress the app-debug.apk file into a <feature>.rar or <feature>.zip file and upload it here]

@@ -1,4 +1,6 @@
<?xml version="1.0" encoding="utf-8"?>


Copy link
Contributor

Choose a reason for hiding this comment

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

Avoid making changes in unrelated files.

@@ -145,6 +145,7 @@ private void buildInstructionDialog() {
.setPositiveButton(R.string.focus_test_msg, (dialog, which) -> {
Intent intent = new Intent(getContext(), DataLoggerActivity.class);
intent.putExtra(ProgramModeActivity.PROGRAM_FLAG_KEY, MemoryGraphParent.MEMORY_GRAPH_FLAG);
intent.addFlags(Intent.FLAG_ACTIVITY_CLEAR_TASK|Intent.FLAG_ACTIVITY_NEW_TASK|Intent.FLAG_ACTIVITY_RESET_TASK_IF_NEEDED);
Copy link
Contributor

Choose a reason for hiding this comment

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

Reformat the code. It will add spaces in between |'s

@jddeep
Copy link
Member

jddeep commented Jun 5, 2020

@htg2704 Any updates on 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.

3 participants