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

Need to fix TECH_NULL in spreadsheet #25

Closed
stackpoint opened this issue Nov 23, 2013 · 5 comments
Closed

Need to fix TECH_NULL in spreadsheet #25

stackpoint opened this issue Nov 23, 2013 · 5 comments
Assignees
Labels
Milestone

Comments

@stackpoint
Copy link
Contributor

I fixed a PrereqTech issue with the Shrine being set to TECH_NULL instead of NULL. However, the code is automatically generated using a google spreadsheet and I don't know how to fix the spreadsheet so that it doesn't generate the TECH_ prefix in front of NULL.

See the following commit for more information: 56efc19

@ghost ghost assigned Thalassicus Nov 23, 2013
@GrantSP
Copy link
Collaborator

GrantSP commented Nov 23, 2013

I'll take that. Should be just a cell reference that needs changing.
On 23/11/2013 3:29 PM, "stackpointer" [email protected] wrote:

I fixed a PrereqTech issue with the Shrine being set to TECH_NULL instead
of NULL. However, the code is automatically generated using a google
spreadsheethttps://docs.google.com/spreadsheet/ccc?key=0Ap8Ehya83q19dDE2TURZSnJSdzE2REx3XzNPY3RVZmcand I don't know how to fix it so that it doesn't generate the TECH_ prefix
in front of NULL.

See the following commit for more information: 56efc1956efc19


Reply to this email directly or view it on GitHubhttps://github.com//issues/25
.

@GrantSP
Copy link
Collaborator

GrantSP commented Nov 23, 2013

Well I was going to but it looks I don't have editing rights on that file.
All we have to do is change the formula in Cell I47 to not add in TECH_
It joins that to the data from Cell C47
It would be good if all the collaborators have access.
On 23/11/2013 3:55 PM, "Grant Pritchard" [email protected] wrote:

I'll take that. Should be just a cell reference that needs changing.
On 23/11/2013 3:29 PM, "stackpointer" [email protected] wrote:

I fixed a PrereqTech issue with the Shrine being set to TECH_NULL instead
of NULL. However, the code is automatically generated using a google
spreadsheethttps://docs.google.com/spreadsheet/ccc?key=0Ap8Ehya83q19dDE2TURZSnJSdzE2REx3XzNPY3RVZmcand I don't know how to fix it so that it doesn't generate the TECH_ prefix
in front of NULL.

See the following commit for more information: 56efc1956efc19


Reply to this email directly or view it on GitHubhttps://github.com//issues/25
.

@stackpoint
Copy link
Contributor Author

You have to join the group: #6 (comment)

@GrantSP
Copy link
Collaborator

GrantSP commented Nov 23, 2013

Thanks. Didn't see that.
On 23/11/2013 4:12 PM, "stackpointer" [email protected] wrote:

You have to join the group: #6 (comment)#6 (comment)


Reply to this email directly or view it on GitHubhttps://github.com//issues/25#issuecomment-29126096
.

@ghost ghost assigned GrantSP Nov 24, 2013
@stackpoint
Copy link
Contributor Author

I'll close this for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants