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

Versão iOS #13

Open
jfoclpf opened this issue Aug 13, 2021 · 21 comments
Open

Versão iOS #13

jfoclpf opened this issue Aug 13, 2021 · 21 comments
Assignees
Labels
enhancement New feature or request

Comments

@jfoclpf
Copy link
Owner

jfoclpf commented Aug 13, 2021

Olá @worm69
Achas que podemos tentar exportar para iOS?
Abraço

@jfoclpf jfoclpf added the enhancement New feature or request label Aug 13, 2021
@jfoclpf jfoclpf assigned jfoclpf and unassigned jfoclpf Aug 13, 2021
@worm69
Copy link
Collaborator

worm69 commented Aug 19, 2021

@jfoclpf a build de ios já funciona mas acho que estou com erro ao enviar as fotos, parece que o pedido é enviado com sucesso mas as fotos não. Podes confirmar no servidor eu depois deixo aqui prints dos testes.
E precisava do icone da app em 1024x1024px para dar fix dos icones em ios

@jfoclpf
Copy link
Owner Author

jfoclpf commented Aug 20, 2021 via email

@worm69
Copy link
Collaborator

worm69 commented Aug 21, 2021

No upload para o servidor
Da uma vista nos prints sff

Captura de ecrã 2021-08-18, às 23 02 38

Captura de ecrã 2021-08-21, às 21 11 19

Captura de ecrã 2021-08-18, às 23 04 22

Captura de ecrã 2021-08-18, às 23 03 16

@jfoclpf
Copy link
Owner Author

jfoclpf commented Aug 21, 2021 via email

jfoclpf added a commit that referenced this issue Aug 21, 2021
@jfoclpf
Copy link
Owner Author

jfoclpf commented Aug 21, 2021

Boas @worm69
podes testar agora?
De facto parece que havia um limite de 1mb para ficheiros, agora pus 50mb, deve ser suficiente :)

@worm69
Copy link
Collaborator

worm69 commented Aug 23, 2021

Sim a noite aproveito e testo as duas apps

@jfoclpf
Copy link
Owner Author

jfoclpf commented Aug 23, 2021

Obrigadão

@worm69
Copy link
Collaborator

worm69 commented Aug 23, 2021

Com uma foto funciona mas com duas da o mesmo erro.

Captura de ecrã 2021-08-23, às 21 03 51

@jfoclpf
Copy link
Owner Author

jfoclpf commented Aug 24, 2021

ok, coloquei como limite 50mb, vou aumentar para 300mb, já deve dar

tenho mesmo de comprimir as fotos do lado do servidor, caso contrário tenho que comprar mais espaço :)

@jfoclpf
Copy link
Owner Author

jfoclpf commented Aug 24, 2021

melhor ainda: https://www.npmjs.com/package/sharp
posso comprimi-las mal são recebidas

@jfoclpf
Copy link
Owner Author

jfoclpf commented Aug 24, 2021

andei aqui às voltas e consegui reproduzir esse bug através do MacInCloud. O problema é que não era apenas do nodejs-express mas também do nginx. Já alterei as limitações, testei e funcionou, consegui fazer o upload das 4 fotos.
Podes testar sff agora @worm69 ?

@jfoclpf
Copy link
Owner Author

jfoclpf commented Aug 25, 2021

testei agora mais uma vez e parece estar a funcionar

screenshot_2

@worm69
Copy link
Collaborator

worm69 commented Aug 26, 2021

Captura de ecrã 2021-08-26, às 21 26 05

No histórico ao tentar abrir as imagens tenho este erro. Alguma ideia? Elas foram envidas com sucesso se não me engano

@jfoclpf
Copy link
Owner Author

jfoclpf commented Aug 27, 2021 via email

@jfoclpf
Copy link
Owner Author

jfoclpf commented Aug 27, 2021

Obrigado @worm69
na realidade descobriste um bug geral :)
Não estava a sanitar o nome do ficheiro e dava barraca quando a freguesia tinha espaços, no teu caso "Rio Frio". Nunca descobri porque nos testes usava apenas as que apareciam logo, ou seja, as primeiras da lista, município Abrantes, freguesia Bemposta, ou seja não têm caracteres especiais nem espaços.

Já está resolvido com 58dadac

Podes agora testar sff?
Mais uma vez obrigado por toda a paciência
um abraço

@jfoclpf
Copy link
Owner Author

jfoclpf commented Aug 27, 2021

screenshot_2

@worm69
Copy link
Collaborator

worm69 commented Sep 15, 2021

O bug dos espaços em branco ja foi resolvido e ja mandei para revisão.

  • precisava do icon da app em 1024x1024 para gerar e configurar o resto dos ícones se tiveres.
    @jfoclpf

@jfoclpf
Copy link
Owner Author

jfoclpf commented Sep 16, 2021

Muito obrigado @worm69
Neste momento estou de férias no Algarve e não tenho acesso ao PC :)
Regresso no Sábado e envio-te logo o ícone
Grande abraço e obrigado

@worm69
Copy link
Collaborator

worm69 commented Sep 16, 2021

Aproveita as ferias com a família abraço

@jfoclpf
Copy link
Owner Author

jfoclpf commented Sep 16, 2021

Obrigado, terminam já no Sábado :)

Um abraço

@jfoclpf
Copy link
Owner Author

jfoclpf commented Sep 19, 2021

  • precisava do icon da app em 1024x1024 para gerar e configurar o resto dos ícones se tiveres.

Olá @worm69 está aqui já na repo: /main/res/icon/universal/1024.png

1 abraço

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

No branches or pull requests

2 participants