Details

    • Type: New Feature
    • Status: Closed
    • Priority: Medium
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Component/s: CLI
    • Labels:
    • Environment:

      Product: Mobile
      Titanium SDK: Mobile CI
      Platform OS: iOS
      Platform OS Version: 5.1

      Description

      Feature Request

      This feature is to add lower case message in our logs as follows:

      logger.error(__('Invalid app id "%s"', cli.argv.id) + '\n');
      logger.log(__('The app id must consist of letters, numbers, and underscores.'));
      logger.log(__('The first character must be a letter or underscore.'));
      logger.log(__("Usually the app id is your company's reversed Internet domain name. (i.e. com.example.myapp)") + '\n');
      

      Suggest is just adding "lower case":

      logger.error(__('Invalid app id "%s"', cli.argv.id) + '\n');
      logger.log(__('The app id must consist of lower case letters, numbers, and underscores.'));
      logger.log(__('The first character must be a letter or underscore.'));
      logger.log(__("Usually the app id is your company's reversed Internet domain name. (i.e. com.example.myapp)") + '\n');
      

      Customer remarks

      "I have no particular reason to have mixed case other than aesthetics. We had just blindly created one with mixed case. It wasn't a problem until I tried a CI sdk and suddenly our app id was invalid, and I had to dig into the code to find out why, lower case is not mentioned as a requirement in the error.

      We are just now getting to the release phase of the iphone version of our app, so maybe we would have run into problems down the road.

      So I'd suggest to specify in the error that lower case is required. It'll probably help others."

        Attachments

          Activity

            People

            • Assignee:
              cbarber Chris Barber
              Reporter:
              egomez Eduardo Gomez (Inactive)
              Reviewer:
              Bryan Hughes
            • Watchers:
              7 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Backbone Issue Sync

                • Backbone Issue Sync is enabled for your project, but we do not have any synchronization info for this issue.

                  Git Integration