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

additional config / folder structure #1

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

Conversation

iamleec
Copy link
Contributor

@iamleec iamleec commented Oct 24, 2022

  1. added core app and AWS dependencies into pubspec.yml replacing the autogenerated default

  2. added assets section into pubspec.yml for Burendo logo along with image folder in root (image to appear in another pull request)

  3. in the folder iOS/Podfile have added a minimum iOS version of 13.5. Anything below this and it will not run.

  4. In android/app I've modified the build.gradle file to run Android SDK 21

  5. added additional folder structure

Copy link

@ChrisScottThomas ChrisScottThomas left a comment

Choose a reason for hiding this comment

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

Added a few fixes for missing new lines. Could you add a README explaining what this is please?

# For details regarding fonts from package dependencies,
# see https://flutter.dev/custom-fonts/#from-packages
## assets:
## - images/burendo.png

Choose a reason for hiding this comment

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

Suggested change
## - images/burendo.png
## - images/burendo.png

900: Color.fromRGBO(255, 207, 68, 1),
};
const MaterialColor primaryColor = MaterialColor(0xFFFFCF44, primarySwatch);
const int primaryColorDark = 0xFFFD9725;

Choose a reason for hiding this comment

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

Suggested change
const int primaryColorDark = 0xFFFD9725;
const int primaryColorDark = 0xFFFD9725;

home,
trip,
edittrip,
}

Choose a reason for hiding this comment

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

Suggested change
}
}

@iamleec iamleec removed the request for review from steveburtonBUR October 27, 2022 08:18
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.

2 participants