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

ps484 #895

Merged
merged 34 commits into from
Sep 10, 2022
Merged

ps484 #895

merged 34 commits into from
Sep 10, 2022

Conversation

Amean
Copy link
Contributor

@Amean Amean commented Aug 26, 2022 •
edited
Loading

Creating a pylons home widget for android that displays user NFTs in a grid

Amean and others added 30 commits August 26, 2022 02:19
@Amean
switched to new repo
f18fc51
@Amean
finish switching to new repo
2ae1a68
@Amean
added package call to provider
68a3d45
@Amean
added package to service
e8f88bc
@Amean
added service to manifest
bcf3670
@Amean
added pigeon dependency
dcbd6f1
@Amean
added message file for flutter android communication interface
cf17e57
@Amean
added message class ad abstract class with method declaration to message
3544054
@Amean
changed nft image type to String and deleted getCollections parameter
2642403
@Amean
placed pigeons dep in the dev dependencies
1887930
@Amean
autogenerated pigeon files
2e2ea0c
@Amean
added collections api call to service
b28477e
@Amean
enabled async on message response and updated class name
eedd3f7
@Amean
updated getCollections call to use correct type
3f9aad2
@Amean
added getCollections implementation
bdf09dc
@Amean
regenerated files for async messages
1a9699e
@Amean
removed home widget usage
cb83bec
@Amean
added missing appwidget provider and intent imports
177a44e
@Amean
added empty view
6a816a1
@Amean
added missing method implementations, passed in binary message to col…
5bd6975 
…lections api, fixed imports
@Amean
fixed imports as well as layout and id calls
70cb9aa
@kjawadDeveloper2
Merge branch 'main' into ps484
1f01c3f
@Amean
added flutter engine that produces binary message
42354c4
@Amean
added image to test
7e765a2
@Amean
set jvm to 11
9665ac8
@Amean
Merge branch 'ps484' of https://github.com/Pylons-tech/pylons into ps484
0464ea4
@Amean
Merge branch 'main' of https://github.com/Pylons-tech/pylons into ps484
b1ba820
@Amean
removed unused home_widget dependency
6e6f989
@Amean
removed unused home widget import
d0a3976
@Amean
added end of line to files
d1a07cc
@Amean Amean marked this pull request as ready for review September 7, 2022 21:53
@faisalnaveedRNS
Copy link
Collaborator

@kjawadDeveloper2 please look in to it

mustafapylons
mustafapylons approved these changes Sep 8, 2022
View reviewed changes
Copy link

@mustafapylons mustafapylons left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@faisalnaveedRNS
Copy link
Collaborator

@kjawadDeveloper2 please review

@faddat faddat merged commit c59d49e into main Sep 10, 2022
@faddat faddat deleted the ps484 branch September 10, 2022 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@faddat faddat faddat approved these changes

@mustafapylons mustafapylons mustafapylons approved these changes

@MikeSofaer MikeSofaer Awaiting requested review from MikeSofaer MikeSofaer is a code owner

@faisalnaveedRNS faisalnaveedRNS Awaiting requested review from faisalnaveedRNS faisalnaveedRNS is a code owner

@kjawadDeveloper2 kjawadDeveloper2 Awaiting requested review from kjawadDeveloper2 kjawadDeveloper2 is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Amean @faisalnaveedRNS @faddat @mustafapylons @kjawadDeveloper2