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

[BUG] clicking on news does not open news source #344

Open
Pydes-boop opened this issue Apr 9, 2024 · 6 comments
Open

[BUG] clicking on news does not open news source #344

Pydes-boop opened this issue Apr 9, 2024 · 6 comments

Comments

@Pydes-boop
Copy link

Describe the bug
Currently when clicking on news in the news tab of the app it does not redirect or open the origin of the news but instead just opens a full view of the preview image.

To Reproduce
Steps to reproduce the behavior:

  1. Go to Home
  2. Scrill down to News
  3. Click on any news

Expected behavior
When clicking on a news article i want to be able to read more context about it or open the source in a browser window.

@CommanderStorm
Copy link
Member

CommanderStorm commented Apr 9, 2024

What news are you clicking on? (can you attach a screenshot)
For some of our news, no browser window exists (the ones whith the blue half of the screen, i.e. "newsspread")

Idk, if there are good solutions to scrape the text from these images and check if some of the contained text is a link => link there..

@CommanderStorm CommanderStorm transferred this issue from TUM-Dev/campus_flutter Apr 9, 2024
@CommanderStorm
Copy link
Member

Transferred to the backend, as this is likely where the issue comes from.

@Pydes-boop
Copy link
Author

Pydes-boop commented Apr 9, 2024

Sorry some additional info:
Right now im on App Version 5.0.1 and im experiencing it when clicking on any news in the tab "latest news"

for example tum thesis printing, but also Talk Food Texture Engineering, Drachenbootrennen and all others i see in the latest news tab.

I think it would be nice if it just opened the source it is referncing at the bottom of the news text.

For the TU Film it works correctly and opens the link when clicking on the different movies.

image

image

@jakobkoerber
Copy link
Member

jakobkoerber commented Apr 9, 2024

I can offer some insights into this: in the beginning we opened the corresponding URL in an In-App-Webview. But after, presumably, changes to the backend the URL was always a link to the password-protected newsspread site. To offer at least some better way to display the information we opted for a full-screen image view. I don't think scraping the image on the client-side would be smart due to performance reasons, we could do that in the backend. If possible, I would opt to bring back public URLs. In general, I was thinking about moving solely to the TUM RSS since these are the only news that affect every student of TUM and are not faculty-specific. But that's another matter 😄

imo this is therefore also not a bug ^^

@CommanderStorm
Copy link
Member

Tum's feed does contain lots of "less related" things (Read: PR).
These screens are a better data source imo

=> lets discuss the approach for news parsing on the next team meeting.

@jakobkoerber
Copy link
Member

These screens are a better data source imo

I think that's debatable but we could think about filtering a bit more depending on the degree of the user. But yes, let's discuss that next meeting :)

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

No branches or pull requests

3 participants