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.

Sign up for GitHub

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

Jump to bottom

Integrate Supabase for Article Fetching, Update Project Images, and Add Social Sharing Functionality #168

Merged
merged 1 commit into from
Jun 21, 2024
Merged

Integrate Supabase for Article Fetching, Update Project Images, and Add Social Sharing Functionality #168

merged 1 commit into from
Jun 21, 2024

Conversation

munezeromicha
Copy link
Contributor

Overview

This pull request introduces several significant enhancements to the article fetching and sharing functionalities in our application. The key features include fetching articles from the Supabase database, updating the article pictures, and enabling users to share articles via social media and messaging applications.

Changes Implemented

  1. Fetching Articles from Supabase Database
  • Integrated Supabase as the backend to fetch articles.
  • Ensured secure and optimized data retrieval processes.
  1. Updating Article Pictures
  • Removed previously used images from the whole project.
  • Implemented a new image handling system for updating project pictures.
  • Ensured the new article images are properly stored and referenced in Supabase.
  • Updated the frontend to display the new images correctly.
  1. Sharing Articles via Social Media and Messaging Applications
  • Added sharing functionality to articles, allowing users to share via popular social media platforms (Facebook, Twitter, LinkedIn, etc.) and messaging apps.
  • Integrated share buttons and modals into the article UI.
  1. ScreenShoots:
Screenshot 1 Screenshot 2 Screenshot 3 Screenshot 4 Screenshot 5 Screenshot 6

@munezeromicha munezeromicha force-pushed the ft-fetch-articles-data branch from 8da132a to 0cce779 Compare June 21, 2024 06:30
@munezeromicha munezeromicha self-assigned this Jun 21, 2024
@munezeromicha munezeromicha added Feature Feature issues Need Review Requires reviews labels Jun 21, 2024
@munezeromicha munezeromicha added this to the Mobile App UI milestone Jun 21, 2024
@munezeromicha munezeromicha linked an issue Jun 21, 2024 that may be closed by this pull request
Fetch Articles Data from Supabase Database #158
Closed
@munezeromicha munezeromicha requested review from mr3nz1, Nkbtemmy, Paccyfic and Irirwanirira and removed request for Paccyfic June 21, 2024 06:31
Irirwanirira
Irirwanirira approved these changes Jun 21, 2024
View reviewed changes
Copy link
Contributor

@Irirwanirira Irirwanirira left a comment

Choose a reason for hiding this comment

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

Good,

@munezeromicha munezeromicha removed the Feature Feature issues label Jun 21, 2024
@munezeromicha munezeromicha removed the request for review from 23nosurrend June 21, 2024 08:20
Nkbtemmy
Nkbtemmy requested changes Jun 21, 2024
View reviewed changes
Copy link
Member

@Nkbtemmy Nkbtemmy left a comment

Choose a reason for hiding this comment

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

This PR has many files changed; Don't repeat it;
changing Images should have been on separate PR !!

.env Outdated
Comment on lines 1 to 5
EXPO_PUBLIC_API_URL=http://api.local
EXPO_PUBLIC_API_KEY=api_key

SUPABASE_URL=https://vbwbfflzxuhktdvpbspd.supabase.co
SUPABASE_NON_KEY=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJzdXBhYmFzZSIsInJlZiI6InZid2JmZmx6eHVoa3RkdnBic3BkIiwicm9sZSI6ImFub24iLCJpYXQiOjE3MTgyNTg0MjgsImV4cCI6MjAzMzgzNDQyOH0.eOPZwYgyg3Ic-jbXW4Rehd2RDgqf-kt4vr30w_O9h3c
Copy link
Member

Choose a reason for hiding this comment

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

First of all; it is not allowed to push .env file

Copy link
Member

Choose a reason for hiding this comment

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

You provide a variable with not values in .env.example

.gitignore
@@ -6,7 +6,7 @@ node_modules/
.expo/
dist/
web-build/
.env
.env.example
Copy link
Member

Choose a reason for hiding this comment

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

why did you change it; it was better how it was

@munezeromicha munezeromicha force-pushed the ft-fetch-articles-data branch from 0cce779 to bad0a5a Compare June 21, 2024 10:32
@munezeromicha munezeromicha requested a review from Nkbtemmy June 21, 2024 10:33
Nkbtemmy
Nkbtemmy requested changes Jun 21, 2024
View reviewed changes
lib/supabase.ts Outdated
"https://vbwbfflzxuhktdvpbspd.supabase.co",
"eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJzdXBhYmFzZSIsInJlZiI6InZid2JmZmx6eHVoa3RkdnBic3BkIiwicm9sZSI6ImFub24iLCJpYXQiOjE3MTgyNTg0MjgsImV4cCI6MjAzMzgzNDQyOH0.eOPZwYgyg3Ic-jbXW4Rehd2RDgqf-kt4vr30w_O9h3c",

export const supabase = createClient('https://vbwbfflzxuhktdvpbspd.supabase.co', 'eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJzdXBhYmFzZSIsInJlZiI6InZid2JmZmx6eHVoa3RkdnBic3BkIiwicm9sZSI6ImFub24iLCJpYXQiOjE3MTgyNTg0MjgsImV4cCI6MjAzMzgzNDQyOH0.eOPZwYgyg3Ic-jbXW4Rehd2RDgqf-kt4vr30w_O9h3c',
Copy link
Member

Choose a reason for hiding this comment

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

This should be in .env

@munezeromicha munezeromicha force-pushed the ft-fetch-articles-data branch from bad0a5a to 681e646 Compare June 21, 2024 10:46
@munezeromicha munezeromicha requested a review from Nkbtemmy June 21, 2024 10:46
@munezeromicha munezeromicha force-pushed the ft-fetch-articles-data branch from 681e646 to 7eff75c Compare June 21, 2024 10:50
Nkbtemmy
Nkbtemmy requested changes Jun 21, 2024
View reviewed changes
lib/supabase.ts Outdated
"https://vbwbfflzxuhktdvpbspd.supabase.co",
"eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJzdXBhYmFzZSIsInJlZiI6InZid2JmZmx6eHVoa3RkdnBic3BkIiwicm9sZSI6ImFub24iLCJpYXQiOjE3MTgyNTg0MjgsImV4cCI6MjAzMzgzNDQyOH0.eOPZwYgyg3Ic-jbXW4Rehd2RDgqf-kt4vr30w_O9h3c",

export const supabase = createClient('https://vbwbfflzxuhktdvpbspd.supabase.co', 'eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJzdXBhYmFzZSIsInJlZiI6InZid2JmZmx6eHVoa3RkdnBic3BkIiwicm9sZSI6ImFub24iLCJpYXQiOjE3MTgyNTg0MjgsImV4cCI6MjAzMzgzNDQyOH0.eOPZwYgyg3Ic-jbXW4Rehd2RDgqf-kt4vr30w_O9h3c',
Copy link
Member

Choose a reason for hiding this comment

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

Why Still here ??

@munezeromicha munezeromicha force-pushed the ft-fetch-articles-data branch from 7eff75c to 1d1cce6 Compare June 21, 2024 10:56
@munezeromicha munezeromicha requested a review from Nkbtemmy June 21, 2024 10:56
@Nkbtemmy Nkbtemmy merged commit 9523da5 into develop Jun 21, 2024
13 checks passed
@Irirwanirira Irirwanirira linked an issue Jun 25, 2024 that may be closed by this pull request
Bug Fix: Resolve environment variables conflicting with expo router. #171
Closed
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Irirwanirira Irirwanirira Irirwanirira approved these changes

@mr3nz1 mr3nz1 mr3nz1 approved these changes

@Nkbtemmy Nkbtemmy Nkbtemmy approved these changes

Assignees

@munezeromicha munezeromicha

Labels
Need Review Requires reviews
Projects
None yet
Milestone
Mobile App UI
4 participants
@munezeromicha @Nkbtemmy @mr3nz1 @Irirwanirira