Skip to content
This repository has been archived by the owner on Oct 23, 2024. It is now read-only.

Adiciona um item irado chamado Monitor de Energia Portatil #126

Merged
merged 11 commits into from
Aug 24, 2024

Conversation

AgentePanela
Copy link

@AgentePanela AgentePanela commented Aug 23, 2024

About the PR

Adiciona o monitor de energia portatil, parecido com o do CMO, o novo tablet do CE gasta 0.75 vezes mais bateria!!

Why / Balance

Agora o CE nao precisa mais levar o computador de energia com ele para ver o que ele quer!!

ideia insana dada pelo @piicat (https://discord.com/channels/1067101251386490991/1271832802345877671)

Copy link

github-actions bot commented Aug 23, 2024

RSI Diff Bot; head commit e0a415b merging into 36daf89
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/EstacaoPirata/Objects/Specific/Engineering/handheldpowermonitor.rsi

State Old New Status
icon Added
scanner Added

Edit: diff updated after e0a415b

@AgentePanela
Copy link
Author

image

Copy link
Collaborator

@DoutorWhite DoutorWhite left a comment

Choose a reason for hiding this comment

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

Pelo que analisei, não vejo nenhum problema de sintaxe, e não parece estar faltando nada, foi inclusive adicionado aos lockers, certamente você se preocupou bastante em deixar tudo funcional. Na próxima vez não precisa colocar tantos comentários, mas tá tudo bem, é opção sua! Só to dizendo que não precisa se preocupar tanto com isso se não quiser, só se for algo realmente complicado de entender só de ler. Enfim, muito obrigado, como sempre! Em QUESTÕES TÉCNICAS não vejo problemas. Parabéns!

@DoutorWhite DoutorWhite merged commit 397395f into rbertoche:pirata Aug 24, 2024
4 of 8 checks passed
@DoutorWhite
Copy link
Collaborator

Tá bem feito e é mais conteúdo pro Pirata, não vejo problemas, o sprite ficou bem legal, por sinal! Se vai ter algum efeito negativo no balanceamento (o que eu duvido, parece que a bateria dura bem pouco e é um objetivo sindie) dá pra descobrir depois com o tempo. Parabéns!

@rbertoche
Copy link
Owner

Eu ia ver esses PR hoje, mas já foram mergidos, mas ainda tenho pedido de alteração.
Na parte da descrição, tirar a parte "magia nanotrasen", esse tipo de coisa vc pode botar no fonte mas não na interface. Magia é coisa de mago, não da nanotrasen.
Coloca algo sobre bluespace no lugar.
Sobre ser um objetivo syndie, não tenho certeza se é bom ou ruim, isso só vendo o desenrolar do jogo, mas tem a vantagem de fazer o CE tentar proteger.
Não testei mas parece legal, parabéns

@AgentePanela

@AgentePanela AgentePanela deleted the energy-monitor branch August 24, 2024 17:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants