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

Payload/aircraft/carriers #17

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Payload/aircraft/carriers #17

wants to merge 3 commits into from

Conversation

5JsAs
Copy link

@5JsAs 5JsAs commented Jun 15, 2020

Config:

  • Added F14 payload
  • Added M2k payload
  • Added gun to AV8B ac
  • Added SC Roosevelt & Lincoln
    Public:
  • Added payload images
  • Added caucacaus airbases for SC Roosevelt & Lincoln

Config:
- Added F14 payload
- Added M2k payload
- Added gun to AV6B ac
- Added SC Roosevelt & Lincoln
Public:
- Added payload images
- Added caucacaus airbases for SC Roosevelt & Lincoln
Comment on lines 3345 to 3370
roosevelt:
name: CVN-71 T.Roosevelt
tacan: 74X
tower: 127.5
approach: 264.0
clearance: 265.0
ground: ICLS 12
departures:
cat: CAT
recoveries:
case1: CASE I
case2: CASE II
case3: CASE III
lincoln:
name: CVN-72 A.Lincoln
tacan: 75X
tower: 128.5
approach: 264.0
clearance: 265.0
ground: ICLS 13
departures:
cat: CAT
recoveries:
case1: CASE I
case2: CASE II
case3: CASE III
Copy link
Owner

Choose a reason for hiding this comment

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

This is special to your mission/template. I cannot merge this in the master. Can you exclude this from the PR please?

Copy link
Author

Choose a reason for hiding this comment

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

Sure

amount: 3
6: *f14station3
7: *f14station2
8a: &f14station8b
Copy link
Owner

Choose a reason for hiding this comment

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

key is 8a, but anchor is 8b?

Also please remove the anchors where they are not needed (as they are not referenced someplace else).

Copy link
Author

Choose a reason for hiding this comment

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

I removed anchor &f14station8b

Copy link
Owner

@obfuscoder obfuscoder left a comment

Choose a reason for hiding this comment

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

Also m2000.PNG should be a lowercase "png" as extension. Linux systems have a case sensitive filesystem.

5JsAs added 2 commits June 22, 2020 11:51
- Removed Roosevelt and Lincoln
- Removed anchor &f14station8b
- PNG to png, but I'm not sure this works in windows
@5JsAs
Copy link
Author

5JsAs commented Jun 22, 2020

Requested changes done, except m2000.PNG to m2000.png. I'm unable to push the file in lowercase extension. Maybe you can.

@obfuscoder
Copy link
Owner

I can change the file name, but this will put me as the author of the change. I can mention you in the commit message if that is ok for you? Also the trailing white space would need to be corrected ...

@5JsAs
Copy link
Author

5JsAs commented Jun 23, 2020

I can change the file name, but this will put me as the author of the change. I can mention you in the commit message if that is ok for you? Also the trailing white space would need to be corrected ...

Not a problem for me, really I don't know how to change PNG uploading from Windows... I should change to Linux.

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